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

138 version switch #143

Closed
wants to merge 2 commits into from
Closed

138 version switch #143

wants to merge 2 commits into from

Conversation

brianknight10
Copy link
Contributor

Attempt to resolve pull request #138 - "--version option invokes help command".

Two simple changes:

  • Added a description of the version switch into the help text.
  • Changed app behavior to simply display "appname version versionno" (e.g., git version 1.8.1) when the --version global switch is used. Previous behavior was to display the help text.

Changed a few cucumber features to reflect the changes. Added a new scenario to test the new feature. All pass, as do tests. All changes are in a "138-version-switch" branch, which should merge into the gli-2 branch as of this date.

@davetron5000
Copy link
Owner

I think the first commit looks good. The second one would be good, if I didn't have a similar version of this on a branch. I've been workingn on some new features on a branch since I wasn't sure if they would pan out. They are and I'm gonna release an RC hopefully today. I'll notify you here when I do

@brianknight10
Copy link
Contributor Author

Cool - thanks for taking the time to review my work. I look forward to seeing the RC. This is a fantastic tool.

@davetron5000
Copy link
Owner

Fixed in 2.6.0 https://github.com/davetron5000/gli/wiki/Changelog thanks for the code!

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