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

Reorder the command line options in the manual #2766

Merged
merged 1 commit into from
Jul 25, 2023

Conversation

itchyny
Copy link
Contributor

@itchyny itchyny commented Jul 25, 2023

I have always thought the order of command line options in the manual is very unfriendly. It lists --seq and --stream at the top, but they are used less often than --null-input and --compact-output. In #2747, I ordered the list carefully to make it easy to understand, and in this PR, I reordered the manual to match the output of --help. Roughly in the order of input options, output options, advanced input/output options (--stream, --seq), module options, arg options, and other options.

@itchyny itchyny added the docs label Jul 25, 2023
@itchyny itchyny added this to the 1.7 release milestone Jul 25, 2023
@itchyny itchyny force-pushed the sort-manual-command-options branch from c96ae26 to f87414d Compare July 25, 2023 00:33
@@ -104,74 +104,58 @@ sections:
You can affect how jq reads and writes its input and output
using some command-line options:

* `--version`/`-V`:
Copy link
Contributor

Choose a reason for hiding this comment

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

You don't think --version should be first?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the option (at least the long option) is obvious and don't think it should be first.

@nicowilliams
Copy link
Contributor

LGTM!

@itchyny
Copy link
Contributor Author

itchyny commented Jul 25, 2023

Thanks.

@itchyny itchyny merged commit 4705a22 into jqlang:master Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants