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

mention that there are many other options, not shown in the default help #3628

Merged
merged 2 commits into from
Nov 28, 2019

Conversation

rdp
Copy link
Contributor

@rdp rdp commented Dec 2, 2016

There are many options "not shown in the default -h" but it doesn't mention them very well.
So add a note so people can know how to get further help, from the command line.

@@ -28,6 +28,8 @@ class Crystal::Command
tool run a tool
help, --help, -h show this help
version, --version, -v show version
Run a command followed with --help to see command specific options, ex:
crystal init --help
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd rather show crystal <command> --help, that's what e.g. git does

@rdp
Copy link
Contributor Author

rdp commented Jun 29, 2017 via email

@@ -28,6 +28,8 @@ class Crystal::Command
tool run a tool
help, --help, -h show this help
version, --version, -v show version
Run a command followed with --help to see command specific options, ex:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe better "followed by"?

@bew
Copy link
Contributor

bew commented Jun 4, 2018

@rdp will you continue working on this?

@rdp
Copy link
Contributor Author

rdp commented Jun 6, 2018 via email

@bew
Copy link
Contributor

bew commented Jun 6, 2018

Well I can't push to your own branch, so I guess I can't just fix it up

@rdp rdp force-pushed the note_more_help_available branch from e3d5817 to 9b1ed02 Compare November 21, 2019 08:08
@rdp
Copy link
Contributor Author

rdp commented Nov 21, 2019

Sorry I missed you message and didn't understand how to accomplish PR's back then. Check it now, better late than never, thank you! :)

@RX14 RX14 added this to the 0.32.0 milestone Nov 28, 2019
@RX14 RX14 merged commit f87c135 into crystal-lang:master Nov 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants