Skip to content

gh-116850: Fix argparse for namespaces with not directly writable dict#124667

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:argparse-namespace-immutable-dict
Sep 29, 2024
Merged

gh-116850: Fix argparse for namespaces with not directly writable dict#124667
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:argparse-namespace-immutable-dict

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Sep 27, 2024

Copy link
Copy Markdown
Member

It now always uses setattr() instead of setting the dict item to modify the namespace. This allows to use a class as a namespace.

…le dict

It now always uses setattr() instead of setting the dict item to modify
the namespace. This allows to use a class as a namespace.
@miss-islington-app

Copy link
Copy Markdown

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

@serhiy-storchaka serhiy-storchaka deleted the argparse-namespace-immutable-dict branch September 29, 2024 08:01
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 29, 2024
…le dict (pythonGH-124667)

It now always uses setattr() instead of setting the dict item to modify
the namespace. This allows to use a class as a namespace.
(cherry picked from commit 95e92ef)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-app

bedevere-app Bot commented Sep 29, 2024

Copy link
Copy Markdown

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 29, 2024
…le dict (pythonGH-124667)

It now always uses setattr() instead of setting the dict item to modify
the namespace. This allows to use a class as a namespace.
(cherry picked from commit 95e92ef)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Sep 29, 2024
@bedevere-app

bedevere-app Bot commented Sep 29, 2024

Copy link
Copy Markdown

GH-124758 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Sep 29, 2024
serhiy-storchaka added a commit that referenced this pull request Sep 29, 2024
…ble dict (GH-124667) (GH-124758)

It now always uses setattr() instead of setting the dict item to modify
the namespace. This allows to use a class as a namespace.
(cherry picked from commit 95e92ef)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
serhiy-storchaka added a commit that referenced this pull request Oct 7, 2024
…ble dict (GH-124667) (GH-124757)

It now always uses setattr() instead of setting the dict item to modify
the namespace. This allows to use a class as a namespace.
(cherry picked from commit 95e92ef)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
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.

1 participant