Skip to content
This repository was archived by the owner on Jun 18, 2026. It is now read-only.

Support args::label for showing better descriptions for types in help text #64

Merged
fasterthanlime merged 7 commits into
bearcove:mainfrom
softdevca:pr-type-description
Feb 5, 2026
Merged

Support args::label for showing better descriptions for types in help text #64
fasterthanlime merged 7 commits into
bearcove:mainfrom
softdevca:pr-type-description

Conversation

@softdevca

Copy link
Copy Markdown
Contributor

Fixes #63

@fasterthanlime

Copy link
Copy Markdown
Contributor

So if you run hooks/install.sh, it'll set up pre-commit and pre-push hooks that would've caught, clippy, docs, and test failures. For this once I'll solve them myself.

@fasterthanlime

Copy link
Copy Markdown
Contributor

To be even more precise, here is what you should have done, either:

  • Checkout main, pull, checkout a fresh branch, cherry-pick the commits you're interested in (without the "doc change to improve --completions help" that I turned down)
  • Stay on the same branch, rebase on top of main, git revert the commit I turned down, then git rebase -i $(git merge-base HEAD origin/main) to squash everything in one commit (or just make a note that they should be squashed when accepting the PR).

Again I'm doing the cleanup this time but 1 PR = 1 clean branch = no merge commits. Only rebases.

@fasterthanlime fasterthanlime added this pull request to the merge queue Feb 5, 2026
@fasterthanlime fasterthanlime removed this pull request from the merge queue due to a manual request Feb 5, 2026
@fasterthanlime fasterthanlime merged commit 84022c9 into bearcove:main Feb 5, 2026
4 checks passed
@github-actions github-actions Bot mentioned this pull request Feb 5, 2026
@softdevca

Copy link
Copy Markdown
Contributor Author

Again I'm doing the cleanup this time but 1 PR = 1 clean branch = no merge commits. Only rebases.

Thanks, will do.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow type name to be specified in help

2 participants