Skip to content

Case-insensitive CLI options#5101

Open
vvolkl wants to merge 2 commits intoeasybuilders:developfrom
vvolkl:case-insensitive-cli-options
Open

Case-insensitive CLI options#5101
vvolkl wants to merge 2 commits intoeasybuilders:developfrom
vvolkl:case-insensitive-cli-options

Conversation

@vvolkl
Copy link

@vvolkl vvolkl commented Jan 30, 2026

The shift key is stuck on my old laptop. Please consider this PR, it will instantly avoid 1.5kg of e-waste!

Comment on lines +155 to +156
_gettext("option %s: invalid choice: %r (choose from %s)")
% (opt, value, choices))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
_gettext("option %s: invalid choice: %r (choose from %s)")
% (opt, value, choices))
_gettext(f"option {opt}: invalid choice: {value} (choose from {choices})")

@smoors
Copy link
Contributor

smoors commented Feb 2, 2026

@vvolkl are you up for adding a test that uses lower case values to test/framework/options.py?

@vvolkl
Copy link
Author

vvolkl commented Feb 3, 2026

@vvolkl are you up for adding a test that uses lower case values to test/framework/options.py?

Sure, let me look into that

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants