Skip to content

Add CLI version lower case -v alias - #1644

Merged
patrickkettner merged 1 commit into
Modernizr:masterfrom
peterblazejewicz:add/version-alias
Sep 12, 2015
Merged

patrickkettner merged 1 commit into
Modernizr:masterfrom
peterblazejewicz:add/version-alias

Conversation

@peterblazejewicz

Copy link
Copy Markdown
Contributor

Most of CLI tools that are known to Modernizr
use lower case options. Let's mention only:
npm -v, node -v, bower -v, brew -v, git -v.
For some reason Modernizr CLI uses uppercase
-V option. This commit provides lower case alias
for now, so uppercase option can be phased out
in the future safely.

The tests:

>> 719 passed! (4.38s)
Done, without errors.

Before this change the well know -v was not supported:

modernizr -v
config file, inline features, or options required.

After this change the -v will work:

modernizr -v
Modernizr v3.0.0

The usage help will output:

modernizr -h
Options:
  -h, --help         Print Help                                                                                      
  -V, -v, --version  Print the version and exit 

Thanks!

Most of CLI tools that are known to Modernizr
use lower case options. Let's mention only:
npm -v, node -v, bower -v, brew -v, git -v.
For some reason Modernizr CLI uses uppercase
-V option. This commit provides lower case alias
for now, so uppercase option can be phased out
in the future safely
patrickkettner added a commit that referenced this pull request Sep 12, 2015
@patrickkettner
patrickkettner merged commit abf300f into Modernizr:master Sep 12, 2015
@patrickkettner

Copy link
Copy Markdown
Member

👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants