Skip to content

Enhance README/lib.rs with advanced command description examples#215

Merged
erickt merged 4 commits into
google:masterfrom
fadeevab:chore/advanced-description
Mar 4, 2026
Merged

Enhance README/lib.rs with advanced command description examples#215
erickt merged 4 commits into
google:masterfrom
fadeevab:chore/advanced-description

Conversation

@fadeevab

@fadeevab fadeevab commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

Improve documentation with the advanced examples of the top-level command "description" (for the help output).
The short variant of a subcommand is reflected in the documentation and the examples.

NOTE: filename examples/goup.rs (GoUp) is chosen to make the help output compelling via the {command_name} placeholder (maybe go-up.rs?)

@fadeevab

Copy link
Copy Markdown
Contributor Author

The advanced description and the Examples section have been supported by argh from day one (I discovered this by reading the code many years ago). However, I believe many people don’t know about it, and this is my attempt to fix that.

@fadeevab

Copy link
Copy Markdown
Contributor Author

@erickt Hi, could you please check this out! Thanks!

@fadeevab fadeevab changed the title chore: add advanced command description and examples to goup Enhance README/lib.rs with advanced command description examples Feb 17, 2026
@fadeevab

fadeevab commented Feb 19, 2026

Copy link
Copy Markdown
Contributor Author

@benbrittain @erickt Thank you for your review in advance!

P.S.: I've already improved the README 6 years ago - PR #16 😄

@erickt erickt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Otherwise looks good, thanks!

Comment thread README.md Outdated
@fadeevab

fadeevab commented Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

@erickt I fixed the warning and cargo test

@erickt erickt merged commit 3065855 into google:master Mar 4, 2026
5 checks passed
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