Skip to content

Conversation

@jbnicolai
Copy link
Contributor

Fixes #188

@jbnicolai
Copy link
Contributor Author

Note that this was the easiest solution, but will now exit without an error code. If this is unacceptable, let me know and I'll refactor display_help()

@tjwebb
Copy link
Collaborator

tjwebb commented Jul 21, 2014

Hmm. While this seems fairly benign at first glace, it would introduce potential breakage for any build script or something that might rely on the error code being consistent.

@jbnicolai
Copy link
Contributor Author

I agree. Updated 😄

@jbnicolai
Copy link
Contributor Author

This should be save to merge now.
// @tjwebb

@maxrimue
Copy link
Contributor

Sorry you had to wait this long, I'll see to get it merged today

@maxrimue maxrimue added this to the 2.1.0 milestone Aug 29, 2015
@maxrimue maxrimue merged commit caa1169 into tj:master Aug 29, 2015
@maxrimue maxrimue removed their assignment Oct 10, 2015
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.

Print help when there are no installed versions

3 participants