Skip to content

Conversation

@wojexe
Copy link

@wojexe wojexe commented Mar 4, 2024

This PR closes issues #633 and #50

✏️ Changes

  • Breaking | Remove the default Swift version and make it optional (swift-version)
  • Read Swift version from a file specified in the action's configuration (swift-version-file)

📝 TODO

  • Add tests

readFile(filePath, "utf8", (err, data) => {
if (err != null) throw err;

const parsedVersion = data.match(semverRegex)?.at(1);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a semver package that's already used in the project that you can use to validate the string 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants