NotificationsYou must be signed in to change notification settings
Fork33.3k
Star69.7k

[3.12] gh-106970: Fix Argument Clinic 'destination <name> clear' command (GH-106972)#106983

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

Jump to bottom

Merged

erlend-aasland merged 1 commit intopython:3.12frommiss-islington:backport-3372bcb-3.12

Jul 22, 2023

Merged

[3.12] gh-106970: Fix Argument Clinic 'destination <name> clear' command (GH-106972)#106983

erlend-aasland merged 1 commit intopython:3.12frommiss-islington:backport-3372bcb-3.12

Jul 22, 2023

Conversation

Copy link

Contributor

miss-islington commentedJul 22, 2023•
edited by bedevere-bot
Loading

Add test for the 'destination clear' command,
and the 'destination' directive in general.

Fix two bugs in 'destination clear' command:

The text attribute of the allocator is called 'text', not '_text'
Return after processing the 'clear' command,
instead of proceeding directly to the fail().
(cherry picked from commit3372bcb)

Co-authored-by: Erlend E. Aaslanderlend@python.org

Issue:Argument Clinic 'destination <name> clear' is broken #106970

pythongh-106970: Fix Argument Clinic 'destination <name> clear' comma…

588f331

…nd (pythonGH-106972)Add test for the 'destination <name> clear' command,and the 'destination' directive in general.Fix two bugs in 'destination <name> clear' command:1. The text attribute of the allocator is called 'text', not '_text'2. Return after processing the 'clear' command,   instead of proceeding directly to the fail().(cherry picked from commit3372bcb)Co-authored-by: Erlend E. Aasland <erlend@python.org>

This was referencedJul 22, 2023

Argument Clinic 'destination <name> clear' is broken#106970

Closed

gh-106970: Fix Argument Clinic 'destination <name> clear' command#106972

Merged