Skip to content

Conversation

@sobolevn
Copy link
Member

@sobolevnsobolevn commented Jul 1, 2024

Please, consider that some subtypes of dict change the signature back to pos-or-keyword args.
But, since this is only about dict, I guess we can be very specific about it.

Do we backport this change?


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

@sobolevnsobolevn requested a review from hugovkJuly 1, 2024 07:16
@bedevere-appbedevere-appbot added docs Documentation in the Doc dir skip news awaiting core review labels Jul 1, 2024
@terryjreedyterryjreedy added needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Jul 1, 2024
@terryjreedy
Copy link
Member

Doc patches usually get backported when applicable, especially if they automerge.

@erlend-aasland
Copy link
Contributor

I would prefer to update the whole dict docs page, not only one method.

@sobolevn
Copy link
MemberAuthor

@erlend-aasland I think that it is complicated. .get is the most complex one. Since Mapping and dict are not the same here.

So, I would prefer to have a smaller scope for this change.

@miss-islington-app
Copy link

Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 1, 2024
…H-121197) (cherry picked from commit 1dc9a4f) Co-authored-by: sobolevn <mail@sobolevn.me>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 1, 2024
…H-121197) (cherry picked from commit 1dc9a4f) Co-authored-by: sobolevn <mail@sobolevn.me>
@bedevere-app
Copy link

GH-121242 is a backport of this pull request to the 3.13 branch.

@sobolevn
Copy link
MemberAuthor

Thanks everyone!

@bedevere-appbedevere-appbot removed the needs backport to 3.13 bugs and security fixes label Jul 1, 2024
@sobolevnsobolevn added needs backport to 3.12 only security fixes and removed needs backport to 3.12 only security fixes labels Jul 1, 2024
@miss-islington-app
Copy link

Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 1, 2024
…H-121197) (cherry picked from commit 1dc9a4f) Co-authored-by: sobolevn <mail@sobolevn.me>
sobolevn added a commit that referenced this pull request Jul 1, 2024
) (#121243) gh-121196: Document `dict.fromkeys` params as pos-only (GH-121197) (cherry picked from commit 1dc9a4f) Co-authored-by: sobolevn <mail@sobolevn.me>
sobolevn added a commit that referenced this pull request Jul 1, 2024
) (#121242) gh-121196: Document `dict.fromkeys` params as pos-only (GH-121197) (cherry picked from commit 1dc9a4f) Co-authored-by: sobolevn <mail@sobolevn.me>
Akasurde pushed a commit to Akasurde/cpython that referenced this pull request Jul 3, 2024
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docsDocumentation in the Doc dirskip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@sobolevn@terryjreedy@erlend-aasland@hugovk