Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-74929: locals() documentation update for PEP 667#118265

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
ncoghlan merged 3 commits intopython:mainfromncoghlan:pep-667-locals-docs
May 6, 2024

Conversation

ncoghlan
Copy link
Contributor

@ncoghlanncoghlan commentedApr 25, 2024
edited by bedevere-appbot
Loading

Proposed documentation wording taken from the
withdrawn PEP 558 (which had switched to the
Python level semantics proposed in PEP 667 prior
to its withdrawal).


📚 Documentation preview 📚:https://cpython-previews--118265.org.readthedocs.build/

Proposed documentation wording taken from thewithdrawn PEP 558 (which had switched to thePython level semantics proposed in PEP 667 priorto its withdrawal).
@ncoghlan
Copy link
ContributorAuthor

DO-NOT-MERGE label added, as this should only be merged after the PEP 667 implementation in#115153 has been completed and merged.

@ncoghlanncoghlan changed the titlelocals() documentation update for PEP 667gh-74929: locals() documentation update for PEP 667Apr 26, 2024
@nineteendo
Copy link
Contributor

Wouldn't it better to convert the pull request back to a draft? GitHub automatically prevents you from merging those.

@ncoghlanncoghlan self-assigned thisApr 28, 2024
@ncoghlanncoghlan marked this pull request as draftApril 28, 2024 10:04
@ncoghlan
Copy link
ContributorAuthor

Wouldn't it better to convert the pull request back to a draft? GitHub automatically prevents you from merging those.

Good point, done (enough years of experience where draft PRs weren't available that I still forget to use them when appropriate)

implicitly refreshed by other operations, such as calling ``locals()``
again. Obtaining the legacy CPython behaviour now requires explicit
calls to update the initially returned dictionary with the results
of subsequent calls to ``locals()``.
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

I plan to eventually replace the latter two sentences here with something like "For details, refer to the 'Python 3.13 porting guide'" (with a link to the relevant part of the 3.13 What's New document). That will be in a later PR that adds an initial version of the What's New update for PEP 667, though.

@ncoghlanncoghlan marked this pull request as ready for reviewMay 6, 2024 10:04
@ncoghlanncoghlan merged commit757fd3e intopython:mainMay 6, 2024
SonicField pushed a commit to SonicField/cpython that referenced this pull requestMay 8, 2024
)Documentation wording taken from thewithdrawn PEP 558 (which had switched to thePython level semantics proposed in PEP 667 priorto its withdrawal).
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees

@ncoghlanncoghlan

Labels
docsDocumentation in the Doc dirskip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@ncoghlan@nineteendo

[8]ページ先頭

©2009-2025 Movatter.jp