-
Notifications
You must be signed in to change notification settings - Fork 543
Update CHANGELOG with 3.10.2 entries #1187
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
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.
LGTM with a version number fix.
Add a section for the new patch release containing the same entries as in the release, e.g. following a `1.2.3` release: | ||
|
||
```md | ||
## 1.3.0 [Unreleased] |
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.
## 1.3.0 [Unreleased] | |
## 1.2.4 [Unreleased] |
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.
1.3.0 is correct because this code block corresponds to the CHANGELOG.md
on master
, not the release branch (as in earlier instructions above).
Do we expect / want to maintain patch release notes on master
prior to patch releases? IIRC we avoid doing so to avoid potential cherry-pick conflicts with the release branch.
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.
On further thought, I believe the expectation is yes (with caveats) given the "Important" note added in #1161 concerning shifting of entries from an unreleased patch section -> next unreleased minor section.
Revised the instructions accordingly.
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.
I believe the expectation is yes
I think so too. The CHANGELOG.md on master
included ## 3.10.2 [Unreleased]
(which this PR updates).
Add a section for the new patch release containing the same entries as in the release, e.g. following a `1.2.3` release: | ||
|
||
```md | ||
## 1.3.0 [Unreleased] |
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.
I believe the expectation is yes
I think so too. The CHANGELOG.md on master
included ## 3.10.2 [Unreleased]
(which this PR updates).
Followup to #1160 which forgot to include CHANGELOG entries in the post-release changes PR due to release instructions only committing the patch entries to the release branch. Instructions are also updated accordingly.