WebSafe 3.7github.com
|
|
🏠
Skip to content

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

Merged
iritkatriel merged 12 commits intopython:mainfrom
savannahostrowski:gh-127133
Nov 24, 2024
Merged

GH-127133: Remove ability to nest argument groups & mutually exclusive groups#127186
iritkatriel merged 12 commits intopython:mainfrom
savannahostrowski:gh-127133

Conversation

@savannahostrowski
Copy link
Member

@savannahostrowski savannahostrowski commented Nov 23, 2024

This PR removes the ability to:

  • Call add_argument_group on an argument group
  • Call add_argument_group or add_mutually_exclusive_group on a mutually exclusive group

We have been warning users of this deprecation since 3.11.


📚 Documentation preview 📚: https://cpython-previews--127186.org.readthedocs.build/

@savannahostrowski
Copy link
Member Author

@iritkatriel Perhaps you can take a look at this one if you've got bandwidth, since you added the original deprecation warnings ☺️

@savannahostrowski
Copy link
Member Author

savannahostrowski commented Nov 23, 2024

Hmm, let me double-check that there aren't other formatting simplifications we can make now as a result of dropping this.

I took a look and ran branch coverage with a couple of different programs, and nothing stood out to me, so I think it's fine to proceed with this and then tackle any other potential refactorings in separate PRs.

@savannahostrowski savannahostrowski marked this pull request as draft November 23, 2024 05:07
@savannahostrowski savannahostrowski marked this pull request as ready for review November 23, 2024 06:06
savannahostrowski and others added 3 commits November 23, 2024 19:09
Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com>
@savannahostrowski savannahostrowski changed the title GH-127133: Remove ability to nest argument groups and mutex groups GH-127133: Remove ability to nest argument groups & mutually exclusive groups Nov 24, 2024
@iritkatriel iritkatriel enabled auto-merge (squash) November 24, 2024 15:20
@iritkatriel iritkatriel merged commit 2104bde into python:main Nov 24, 2024
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

Comments