Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow prerelease versions in release headers #130

Merged
merged 1 commit into from Nov 15, 2022
Merged

Conversation

mcmire
Copy link
Contributor

@mcmire mcmire commented Nov 15, 2022

Currently, this tool expects the version in a release header to be a version that looks like <major>.<minor>.<patch> and rejects other formats. However, it may be useful to publish a prerelease version of a package so that integration with other products/services can be tested in a more "live" manner, particularly if that package is a part of a monorepo. A prerelease version looks like
<major>.<minor>.<patch>-<prerelease identifier>.<number>, where <prerelease identifier> can be an arbitrary word like alpha, beta, or rc. This commit allows such versions.

Currently, this tool expects the version in a release header to be a
version that looks like `<major>.<minor>.<patch>` and rejects other
formats. However, it may be useful to publish a prerelease version of a
package so that integration with other products/services can be tested
in a more "live" manner, particularly if that package is a part of a
monorepo. A prerelease version looks like
`<major>.<minor>.<patch>-<prerelease identifier>.<number>`, where
`<prerelease identifier>` can be an arbitrary word like `alpha`, `beta`,
or `rc`. This commit allows such versions.
@mcmire mcmire requested a review from a team as a code owner November 15, 2022 19:02
@mcmire
Copy link
Contributor Author

mcmire commented Nov 15, 2022

This commit should hopefully fix the CI errors we're seeing here: https://github.com/MetaMask/controllers/actions/runs/3473161747/jobs/5804878754#step:6:14

Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@mcmire mcmire merged commit dcf3e3d into main Nov 15, 2022
@mcmire mcmire deleted the allow-prerelease-versions branch November 15, 2022 21:41
@mcmire mcmire mentioned this pull request Nov 15, 2022
legobeat pushed a commit to legobeat/auto-changelog that referenced this pull request Dec 13, 2022
Currently, this tool expects the version in a release header to be a
version that looks like `<major>.<minor>.<patch>` and rejects other
formats. However, it may be useful to publish a prerelease version of a
package so that integration with other products/services can be tested
in a more "live" manner, particularly if that package is a part of a
monorepo. A prerelease version looks like
`<major>.<minor>.<patch>-<prerelease identifier>.<number>`, where
`<prerelease identifier>` can be an arbitrary word like `alpha`, `beta`,
or `rc`. This commit allows such versions.
@legobeat legobeat mentioned this pull request Dec 13, 2022
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.

None yet

3 participants