Skip to content

MAINT: ensure towncrier can be run >1x, and is included in spin docs #25933

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 2 commits into from
Mar 5, 2024

Conversation

rgommers
Copy link
Member

@rgommers rgommers commented Mar 4, 2024

This allows accumulating snippets into the main release notes file and committing the result during a release cycle. It will also make Sphinx formatting errors, cross-link issues, etc. visible when docs get built locally. This will allow fixing them as we go, rather than leaving it broken in devdocs and then having to fix it in one go once towncrier is run once right before or after creating a release branch.

xref gh-25921 and gh-25827 for recent issues with this.

@rgommers
Copy link
Member Author

rgommers commented Mar 4, 2024

@charris
Copy link
Member

charris commented Mar 4, 2024

I am not clear on how notes-towncrier.rst is supposed to be used.

@mattip
Copy link
Member

mattip commented Mar 4, 2024

I think this is as good as it gets while using towncrier. There is some cruft left at the bottom of the release notes but after branching the notes are tweaked anyway.

@charris
Copy link
Member

charris commented Mar 4, 2024

Note that the release notes also require .. currentmodule:: numpy on the first line when the release happens.

This allows accumulating snippets into the main release notes file and
committing the result during a release cycle. It will also make Sphinx
formatting errors, cross-link issues, etc. visible when docs get built
locally. This will allow fixing them as we go, rather than leaving it
broken in devdocs and then having to fix it in one go once `towncrier`
is run once right before or after creating a release branch.

xref numpygh-25921 and numpygh-25827 for recent issues with this.

[skip azp] [skip actions] [skip cirrus]
@rgommers rgommers force-pushed the fix-towncrier-invocation branch from 3018fb1 to f90d3c5 Compare March 5, 2024 07:35
@rgommers
Copy link
Member Author

rgommers commented Mar 5, 2024

Note that the release notes also require .. currentmodule:: numpy on the first line when the release happens.

Fixed that in doc/source/release/template.rst so it cannot be forgotten about anymore.

I also updated the release walkthrough document.

Copy link
Member

@mattip mattip left a comment

Choose a reason for hiding this comment

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

LGTM, one small question about the RELEASE_WALKTHROUGH.

Copy link
Member

@seberg seberg left a comment

Choose a reason for hiding this comment

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

Thanks, nice to just always build it and it doesn't seem to make the release process harder.
I still don't like the idea of committing partial changes except maybe very late in the release cycle, but it doesn't matter.

In either case @charris is the one who dealt with this the most in the past and currently, so I think he should sign off/merge.


New Features
============
*We'll choose highlights for this release near the end of the release cycle.*
Copy link
Member

Choose a reason for hiding this comment

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

We actually have a highlights section also in the release notes fragments which I think makes sense to not forget a highlight. OTOH, I am not sure we ever actually used it so this is fine also.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah I think it's easier to determine at the end. It's a bit weird to add a note to one's own PR that the thing you're working on right now will be a highlight of the next release:)

Copy link
Member

Choose a reason for hiding this comment

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

Sure, you certainly have to audit it at the end.

@charris charris merged commit 8b0bc6d into numpy:main Mar 5, 2024
@charris
Copy link
Member

charris commented Mar 5, 2024

Well, let's give it a shot. Thanks Ralf.

@rgommers rgommers deleted the fix-towncrier-invocation branch March 5, 2024 15:16
@rgommers rgommers added this to the 2.0.0 release milestone Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants