-
Notifications
You must be signed in to change notification settings - Fork 115
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
Makefile: Add tag-next-release target #2550
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2550 +/- ##
==========================================
- Coverage 67.04% 66.91% -0.13%
==========================================
Files 328 328
Lines 30262 30262
==========================================
- Hits 20289 20250 -39
- Misses 7448 7483 +35
- Partials 2525 2529 +4
Continue to review full report at Codecov.
|
@@ -1 +0,0 @@ | |||
runtime: Extract common sealing code. |
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.
Was this forgotten? How did that happen?
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.
So, I ran make changelog
on the master
branch before #2537 was merged and submitted the Change Log for 20.0 PR (#2540).
Since #2537 was merged before the Change Log PR and I saw that the change was trivial, i.e. it didn't impact the Change Log, I just rebased my PR on top of it. But this then didn't delete the 2537.trivial.md
fragment 🙂.
tag-next-release: | ||
@$(ENSURE_NEXT_VERSION) | ||
@$(ECHO_STDERR) "Checking if we can tag version $(NEXT_VERSION) as the next release..." | ||
@$(ENSURE_NO_CHANGELOG_FRAGMENTS) |
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.
Ah probably it was due to this missing check :-)
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.
Yes, this check will ensure this can't happen in the future 🙂.
It was mistakenly left in when preparing the Change Log for release 20.0: #2540.
Add a bunch of helpers to common.mk to allow re-use. Refactor changelog target to use the new helpers.
This should make it easier to test the release tooling without affecting the real releases.
b1cd0a3
to
63f7cb9
Compare
Closes: #2458.