Skip to content

Improve fish completions#911

Merged
natecook1000 merged 7 commits into
apple:mainfrom
rgoldberg:improve-fish-completions
Jun 4, 2026
Merged

Improve fish completions#911
natecook1000 merged 7 commits into
apple:mainfrom
rgoldberg:improve-fish-completions

Conversation

@rgoldberg
Copy link
Copy Markdown
Collaborator

@rgoldberg rgoldberg commented May 27, 2026

Simple improvements for fish completions.

I have more improvements pending for a subsequent PR, but I haven't looked at them in a while, so need to test those more, while these are ready to go.

I figured I should submit these simple improvements to merge them without waiting for the more complex improvements.

Checklist

  • I've added at least one test that validates that my change is working, if appropriate
  • I've followed the code style of the rest of the project
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary

@rgoldberg rgoldberg force-pushed the improve-fish-completions branch from 4a02915 to fc6756c Compare May 28, 2026 20:57
rgoldberg added 7 commits May 31, 2026 20:43
Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
…ipts.

Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
…ipts more concise.

Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
…ouldOfferCompletionsForPositionalFunction.

Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
@rgoldberg rgoldberg force-pushed the improve-fish-completions branch from 561d0aa to 5114da9 Compare June 1, 2026 00:44
@natecook1000 natecook1000 merged commit 2c37ef8 into apple:main Jun 4, 2026
34 checks passed
@rgoldberg rgoldberg deleted the improve-fish-completions branch June 4, 2026 15:51
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