Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@HansLehnert
Copy link
Contributor

Fix completions for Fish shell when the value being completed has spaces
or is enclosed in quotes

Fixes #2995

@Rowlando13 Rowlando13 changed the base branch from main to stable August 6, 2025 07:00
@Rowlando13
Copy link
Collaborator

Can you please rebase so that it only contains your commit. I changed the branch to stable.

@Rowlando13
Copy link
Collaborator

@AndreasBackx Would you mind reviewing this?

@Rowlando13 Rowlando13 added the bug label Aug 6, 2025
@HansLehnert
Copy link
Contributor Author

I rebased the changes

@AndreasBackx AndreasBackx merged commit 701b313 into pallets:stable Aug 9, 2025
10 checks passed
@AndreasBackx
Copy link
Collaborator

Looks sane, I'm not super familiar with Fish itself and how it handles the COMP_CWORD. I'm more familiar with bash/zsh but this doesn't seem wrong and it does fix the issue. Thanks @HansLehnert!

@kdeldycke kdeldycke added this to the 8.2.3 milestone Aug 12, 2025
@Rowlando13
Copy link
Collaborator

@HansLehnert could you please make a separate PR for changes. I did not see there wasn't one.

@AndreasBackx
Copy link
Collaborator

I'll take care of the CHANGES mention, don't worry about it as I merged this without checking. @HansLehnert

@Rowlando13 Rowlando13 modified the milestones: 8.2.3, 8.3.0 Aug 24, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect handling of quoted & escaped parameters for Fish completions

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.