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

gh-124872: Move PyThreadState to first argument for consistency by rhansen · Pull Request #124774 · python/cpython · GitHub

Repository navigation

gh-124872: Move PyThreadState to first argument for consistency - #124774

Merged
1st1 merged 1 commit into
python:mainfrom
rhansen:gh-119333-notify
Oct 12, 2024
Merged

1st1 merged 1 commit into
python:mainfrom
rhansen:gh-119333-notify

Conversation

rhansen commented Sep 29, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

This is a trivial readability tweak, I don't think a NEWS blurb is warranted; the existing blurb should suffice.

bedevere-app Bot commented Sep 29, 2024

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

bedevere-app Bot commented Oct 1, 2024

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

rhansen changed the title gh-119333: Move PyThreadState to first argument for consistency gh-124872: Move PyThreadState to first argument for consistency Oct 1, 2024

1st1 commented Oct 9, 2024

Copy link
Copy Markdown
Member

cc @fried

1st1 merged commit 62d5a53 into python:main Oct 12, 2024
rhansen deleted the gh-119333-notify branch October 12, 2024 19:49
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL