FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.11] gh-106970: Fix Argument Clinic 'destination <name> clear' command (#106972) by erlend-aasland · Pull Request #107059 · python/cpython · GitHub

/ cpython Public

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

Merged
erlend-aasland merged 1 commit into
python:3.11from
erlend-aasland:backport-3.11-clinic-dest-clear-fix
Jul 22, 2023
Merged

[3.11] gh-106970: Fix Argument Clinic 'destination <name> clear' command (#106972)#107059
erlend-aasland merged 1 commit into
python:3.11from
erlend-aasland:backport-3.11-clinic-dest-clear-fix

Conversation

erlend-aasland commented Jul 22, 2023
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

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

Fix two bugs in 'destination 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 commit 3372bcb)

…' command (python#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 commit 3372bcb)

Copy link
Copy Markdown
Contributor Author

FTR, test_unknown_destination_command() was accidentally added in gh-107002. Originally it was a part of gh-106972, so it should have been a part of this backport as well.

erlend-aasland enabled auto-merge (squash) July 22, 2023 17:25
erlend-aasland merged commit 63c945a into python:3.11 Jul 22, 2023
erlend-aasland deleted the backport-3.11-clinic-dest-clear-fix branch July 22, 2023 23:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL