-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
feat(bumpVersion): add prerelease semver level #15626
feat(bumpVersion): add prerelease semver level #15626
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test versions should match fixture
The bumpVersion feature is driven by the the npm semver dependency which supports the prerelease semver level. Enabling prerelease bumpVersion can lead to version increments like the following: ``` - <version>1.0.0-1</version> + <version>1.0.0-2</version> <dependencies> <dependency> <groupId>org.postgresql</groupId> <artifactId>postgresql</artifactId> - <version>42.3.4</version> + <version>42.3.5</version> </dependency> </dependencies> ``` More details of the initial discussion for this feature can be found here: renovatebot#15573
675d5f3
to
a825e3b
Compare
The force push in my fork adds the recommended changes:
It also rebases against |
please don't force push next time, use merge commit instead. we'll squash the PR so don't worry about history. for e push sometimes need a full new review, as all files are marked as changed, so it'll defer PR merge because of more work. |
Lint error:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs lint fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs fixes
Thanks for the follow up. Seems the latest content caused the linting errors. Prior to pulling the content, I put in a fix. The linter seems to be passing locally now:
|
I suspect the test cases will pass too:
|
🎉 This PR is included in version 32.98.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
The bumpVersion feature is driven by the the npm semver dependency which supports the prerelease semver level.
Enabling prerelease bumpVersion can lead to version increments like the following:
More details of the initial discussion for this feature can be found here:
#15573
Context
Only
major
,minor
andpatch
are currently supported bumpVersion semver levels. The npm semver dependency which drives the bump mechanism also supportsprerelease
. In my case, this would be a useful feature for managing and distributing multiple maven dependencies between repos.Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via:
Note: Some unit tests did not run on my M1 Mac due to
JavaScript heap out of memory
. They will be reran anyways with the GitHub action.Repo configuration:
Executing the test:
Resulting PR: