Thanks to visit codestin.com
Credit goes to github.com

Skip to content

GH-127133: Remove ability to nest argument groups & mutually exclusive groups #127186

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
iritkatriel merged 12 commits into from
Nov 24, 2024
Merged

GH-127133: Remove ability to nest argument groups & mutually exclusive groups #127186

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
e9bf875
Remove functionality and update docs
savannahostrowski Nov 23, 2024
41464a5
Change error type
savannahostrowski Nov 23, 2024
b9f3e4d
📜🤖 Added by blurb_it.
blurb-it[bot] Nov 23, 2024
76aadc6
Add PR number
savannahostrowski Nov 23, 2024
17556c0
Merge branch 'main' into gh-127133
savannahostrowski Nov 23, 2024
1673277
Fix sphinx
savannahostrowski Nov 23, 2024
baed27b
Fix refs in whatsnew
savannahostrowski Nov 23, 2024
813a2f5
Fix references in news entry
savannahostrowski Nov 23, 2024
6c837ab
precommit
savannahostrowski Nov 23, 2024
d835745
Apply suggestions from code review
savannahostrowski Nov 24, 2024
9708536
Reframe docs to focus on exception
savannahostrowski Nov 24, 2024
c55dbee
Fix test exception messages
savannahostrowski Nov 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix test exception messages
  • Loading branch information
@savannahostrowski
savannahostrowski committed Nov 24, 2024
commit c55dbee846165f2774ebcedcc7c4b788e6abd0e5
4 changes: 2 additions & 2 deletions Lib/test/test_argparse.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3001,7 +3001,7 @@ def test_nested_argument_group(self):
parser = argparse.ArgumentParser()
g = parser.add_argument_group()
self.assertRaisesRegex(ValueError,
'nested argument groups are not supported',
'argument groups cannot be nested',
g.add_argument_group)

# ===================
Expand Down Expand Up @@ -3309,7 +3309,7 @@ def test_nested_mutex_groups(self):
g = parser.add_mutually_exclusive_group()
g.add_argument("--spam")
self.assertRaisesRegex(ValueError,
'nested mutually exclusive groups are not supported',
'mutually exclusive groups cannot be nested',
g.add_mutually_exclusive_group)

class MEMixin(object):
Expand Down