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

move --help from stderr to stout #2105

Closed
ondohotola opened this issue Sep 4, 2024 · 6 comments · Fixed by #2138
Closed

move --help from stderr to stout #2105

ondohotola opened this issue Sep 4, 2024 · 6 comments · Fixed by #2138
Assignees
Labels
enhancement New feature or request. Once marked with this label, its in the backlog.

Comments

@ondohotola
Copy link

If I run --help I have to add 2>&1 before I can pipe it through less which I don't have to do for --list.

I often forget and then have to quit less, repeat the command, with the redirection.

@jqnatividad jqnatividad added the enhancement New feature or request. Once marked with this label, its in the backlog. label Sep 5, 2024
@jqnatividad
Copy link
Collaborator

This requires tweaking docopt a bit.

Added it to the backlog @ondohotola .

@ondohotola
Copy link
Author

Thanks.

@jqnatividad
Copy link
Collaborator

To confirm, looking at GNU guidelines...

https://www.gnu.org/prep/standards/standards.html#g_t_002d_002dhelp

It does specify that --help and --version should both go to stdout, not stderr.

@jqnatividad jqnatividad self-assigned this Sep 12, 2024
@jqnatividad
Copy link
Collaborator

jqnatividad commented Sep 12, 2024

Moved qsv_docopt crate to https://github.com/dathere/qsv-docopt.

Fixes will be done there going forward.

@rzmk, also will implement the enhancement there so qsv pro can show usage text. Thinking of using markdown on all usage text so it looks prettier on qsv pro...

@rzmk
Copy link
Collaborator

rzmk commented Sep 12, 2024

If switching to markdown for the usage text then it may be neat to render it as formatted in the terminal too:

@jqnatividad
Copy link
Collaborator

termimad looks awesome @rzmk!

Apart from styling, it handles scrolling too!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request. Once marked with this label, its in the backlog.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants