FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Sample text for closing future stdlib PRs by srittau · Pull Request #13697 · python/typeshed · GitHub

Repository navigation

Sample text for closing future stdlib PRs - #13697

Merged
srittau merged 6 commits into
python:mainfrom
srittau:maintainer-guide-future-stdlib
Mar 27, 2025
Merged

srittau merged 6 commits into
python:mainfrom
srittau:maintainer-guide-future-stdlib

Conversation

srittau commented Mar 21, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread MAINTAINERS.md Outdated
Comment thread MAINTAINERS.md Outdated
Comment thread MAINTAINERS.md Outdated
When rejecting a PR for a change for a future Python version, use a message
like:

Thanks for contributing! Unfortunately, [as outlined in our CONTRIBUTING document](https://github.com/python/typeshed/blob/main/CONTRIBUTING.md#standard-library-stubs) we only accept pull requests to the standard library for future Python versions after the first beta version has been released. This is in part to prevent churn in the stubs, and in part because the testing infrastructure for the future version is not yet in place. When the first beta version has been released and this PR is still relevant, please feel free to leave a comment here to reopen it. Alternatively, you can open a new PR.

Avasam Mar 27, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Alternatively, you can open a new PR.

I initially read this as

Alternatively (to leaving a comment later), you can open a new PR. (now)

but I'm not sure how to make it any clearer without being overly verbose.


Other than that it seems like a perfectly fine message to me.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I've switched the two alternatives around. Maybe that is clearer?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Looks much clearer to me 👍

srittau merged commit 3890289 into python:main Mar 27, 2025
srittau deleted the maintainer-guide-future-stdlib branch March 27, 2025 16:46
mmingyu pushed a commit to mmingyu/typeshed that referenced this pull request May 16, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL