Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Nov 13, 2025

@encukou
Copy link
Member

encukou commented Nov 14, 2025

You might also want to add a note to Construction that you can do frozendict(collection, **additional_kwargs).

vstinner and others added 4 commits November 14, 2025 11:41
@vstinner vstinner marked this pull request as ready for review November 14, 2025 17:57
@vstinner vstinner requested a review from corona10 as a code owner November 14, 2025 17:57
@vstinner
Copy link
Member Author

@corona10: Would you mind to review these changes?

Copy link
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

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

lgtm!

@vstinner vstinner merged commit daa024d into python:main Nov 15, 2025
5 checks passed
@vstinner vstinner deleted the pep814_update branch November 15, 2025 03:54
@efimov-mikhail
Copy link
Member

efimov-mikhail commented Nov 15, 2025

There's a typo frozencopy.copy() -> frozendict.copy().
CC @vstinner

I can make a PR with this "one line change", if you want to.

@vstinner
Copy link
Member Author

There's a typo frozencopy.copy() -> frozendict.copy().

Oops. I wrote #4708 to prepare the next batch of changes :-) I fixed the typo there. Thanks for reporting the typo.

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.

5 participants