-
-
Notifications
You must be signed in to change notification settings - Fork 536
Fix custom HelpFormatter for Python 3.14 #3524
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
python/cpython#132323 passes prefix_chars= (and other arguments) to the formatter_class, but the custom HelpFormatter only accepted prog=. Accept any keyword arguments and pass them on to the parent class. Fixes tox-dev#3523
RTD build failure is unrelated: File "/home/docs/checkouts/readthedocs.org/user_builds/tox/checkouts/3524/.tox/docs/lib/python3.13/site-packages/snowballstemmer/__init__.py", line 27, in stemmer
raise KeyError("Stemming algorithm '%s' not found" % lang)
KeyError: "Stemming algorithm 'porter' not found" Test failures seem unrelated too:
|
GitHub Actions supports 3.14 beta 1 (which is why I saw this error in wntrblm/nox#970), so it could be added to the CI. |
The snowball thing is yanked, so that should be fine now. The wheel change is something separate that needs fixing in the tests, probably related to virtualenv. |
I'm also seeing this locally:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, cnfirmed this PR fixes tox on 3.14.
Error before:
https://github.com/hugovk/cherry-picker/actions/runs/14954301375/job/42007915084
Passes with this PR:
https://github.com/hugovk/cherry-picker/actions/runs/14954355501/job/42008033162
(Windows failure is unrelated: python/cpython#133779)
python/cpython#132323 passes prefix_chars= (and other arguments) to the formatter_class, but the custom HelpFormatter only accepted prog=.
Accept any keyword arguments and pass them on to the parent class.
Fixes #3523
tox -e fix
)docs/changelog
folder