Skip to content

DOC: update release process maintainer guide #61810

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jorisvandenbossche
Copy link
Member

A small things I noticed that could use some update or clarification while releasing 2.3.1.

cc @mroeschke can you check that this confirms with your experience when releasing 2.3.0?

@simonjayhawkins
Copy link
Member

@jorisvandenbossche one thing that I used to do during pre-release is ensure that the release notes between the backport and the main branch were in sync.

Discrepancies can arise if backports are missed or merge conflicts on the backport were perhaps not resolved correctly, so this used to be a beneficial check.

IIRC we had some issues with the 2.3.0 release that would have been picked up if this was done. As well as the release process instructions, I used to use a release checklist on the release issue.

Maybe, while updating the process, you could add something to "Update and clean release notes for the version to be released, including:" section?

4. If not a release candidate, make sure all backporting pull requests to the
branch being released are merged, and no merged pull requests are missing a
backport (check the
["Still Needs Manual Backport"](https://github.com/pandas-dev/pandas/labels/Still%20Needs%20Manual%20Backport)
Copy link
Member

Choose a reason for hiding this comment

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

Just noting this markdown style linking won't hyperlink as expected since this is rst.

Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

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

Thanks, yes the new limitations you added here were my experience as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants