Skip to content
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

Parsing of Boolean values should be case insensitive #24

Closed
benpeart opened this issue Nov 12, 2020 · 3 comments
Closed

Parsing of Boolean values should be case insensitive #24

benpeart opened this issue Nov 12, 2020 · 3 comments
Labels
enhancement New feature or request

Comments

@benpeart
Copy link

When specifying boolean environment variables or command line options, traditionally they are all lower case. When I attempted this, it didn't work and it took me a while to figure out I had to do mixed case. Making them case insensitive would be more flexible and less error prone.

@johanmeijer
Copy link
Owner

The mixed case for boolean (first character uppercase) is a python naming convention. I do understand this can be confusing if you have to specify this in the parameters.

I will try to improve this in the next releases. Thanks.

@johanmeijer johanmeijer added the enhancement New feature or request label Nov 12, 2020
@johanmeijer
Copy link
Owner

Versie 2.3.0 (now available in the beta branche) also allow lower-case boolean in .ini and environmental variables.

@johanmeijer
Copy link
Owner

Changes are generic available in master branche.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants