Skip to content

Allow prerelease versions in release headers#130

Merged
mcmire merged 1 commit intomainfrom
allow-prerelease-versions
Nov 15, 2022
Merged

Allow prerelease versions in release headers#130
mcmire merged 1 commit intomainfrom
allow-prerelease-versions

Conversation

@mcmire
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

3 participants