Skip to content

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

Merged
merged 4 commits into from
Aug 6, 2024

Conversation

eramongodb
Copy link
Contributor

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.

@eramongodb eramongodb requested a review from kevinAlbs August 6, 2024 18:17
@eramongodb eramongodb self-assigned this Aug 6, 2024
Copy link
Collaborator

@kevinAlbs kevinAlbs left a 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]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
## 1.3.0 [Unreleased]
## 1.2.4 [Unreleased]

Copy link
Contributor Author

@eramongodb eramongodb Aug 6, 2024

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.

Copy link
Contributor Author

@eramongodb eramongodb Aug 6, 2024

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.

Copy link
Collaborator

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).

@eramongodb eramongodb requested a review from kevinAlbs August 6, 2024 18:50
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]
Copy link
Collaborator

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).

@eramongodb eramongodb merged commit 289c250 into mongodb:master Aug 6, 2024
54 of 60 checks passed
@eramongodb eramongodb deleted the cxx-changelog branch August 6, 2024 19:12
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.

2 participants