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

CLI options: improve handling of booleans and other cases #914

Merged

ecraig12345 merged 1 commit into master from ecraig/cli-option-types

Sep 1, 2023

Member

ecraig12345 commented Sep 1, 2023

Ensure that that text values true and false are handled as booleans
Add explicit expected option types for all known CLI options
Refine the returned value to not include camel case or aliased keys
Add tests for getCliOptions (this will help ensure predictable behavior with future yargs-parser upgrades or if we change to another parser like commander)


          Ensure getCliOptions properly handles all boolean and numeric options

c8710e2

ecraig12345 merged commit 6223ecb into master

4 checks passed

ecraig12345 deleted the ecraig/cli-option-types branch

September 1, 2023 03:10

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

Labels

None yet