Skip to content
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

Move NEWS entries to pull-request-lifecycle #1506

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

StanFromIreland
Copy link
Contributor

@StanFromIreland StanFromIreland commented Feb 3, 2025

@StanFromIreland
Copy link
Contributor Author

StanFromIreland commented Feb 3, 2025

@StanFromIreland
Copy link
Contributor Author

See also would be nice but it seems to make the section quoted.

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

See also would be nice but it seems to make the section quoted.

How do you mean?

@StanFromIreland
Copy link
Contributor Author

image

I must have done something weird before it works when I tried again!

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Thanks!

@StanFromIreland StanFromIreland requested a review from picnixz March 13, 2025 15:38
Copy link
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

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

We improved a bit the content, but not by much, so it's fine to make it part of this change. I think we need a follow-up PR to improve the cases where a NEWS entry is necessary (see python/cpython#131177 (comment)). OTOH, you can revert the 3.X.rst mention and make it part of the follow-up PR.

Before merging, I'll check that the rst:role is correctly linked though.

@picnixz
Copy link
Member

picnixz commented Mar 13, 2025

Arf, it appears that we don't have the cross-referencing:

image

Well, let's introduce intersphinx in a future PR.

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.

Consider relocating NEWS/What's New docs
3 participants