8000 Fish completions no longer suggest subcommands that have already been picked by bittrance · Pull Request #2117 · urfave/cli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fish completions no longer suggest subcommands that have already been picked #2117

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

Merged
merged 1 commit into from
Apr 27, 2025

Conversation

bittrance
Copy link
Contributor

What type of PR is this?

  • bug

What this PR does / why we need it:

Stop fish completions suggesting sub-commands from the level you just picked from.

Which issue(s) this PR fixes:

Fixes #2116 .

Special notes for your reviewer:

This solution tries to minimize the size of this change. It is possible and arguably easier to read to use multiple -n which will be and:ed together for you.

Testing

See #2116 for test code.

Release Notes

Stop fish completions suggesting sub-commands from the level you just picked from.

@bittrance bittrance requested a review from a team as a code owner April 27, 2025 15:00
@bittrance
Copy link
Contributor Author

The refactored helpers in this PR will need to be further extended along the same lines to include the lineage of a sub-command in order to address #2115.

@dearchap dearchap merged commit 5357b6d into urfave:main Apr 27, 2025
10 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.

v3 fish completions suggest already selected command for L2+ commands
2 participants
0