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

Jump to bottom

Remove required argument from parser methods #403

Merged

sloria merged 1 commit into main from rm-required

Jan 23, 2025

Owner

sloria commented Jan 23, 2025 •

edited

Loading

required is redundant with not passing a default value to parser methods.

it was never documented so arguably not a breaking change, but marking it as such anyway in the changelog in case anyone found it on accident.


          Remove required argument from parser methods

3ff6676

sloria merged commit f460dbe into main

10 checks passed

sloria deleted the rm-required branch

January 23, 2025 02:47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet