Skip to content

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

Merged
dearchap merged 1 commit intourfave:mainfrom
bittrance:fish-completion-repetition
Apr 27, 2025
Merged

Fish completions no longer suggest subcommands that have already been picked#2117
dearchap merged 1 commit intourfave:mainfrom
bittrance:fish-completion-repetition

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