-
Notifications
You must be signed in to change notification settings - Fork 204
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
docs: improve the fish completions location #1647
docs: improve the fish completions location #1647
Conversation
docs/index.md
Outdated
@@ -78,7 +78,7 @@ Add-Content -Path $PROFILE -Value '(& pixi completion --shell powershell) | Out- | |||
### Fish | |||
|
|||
```fish | |||
echo 'pixi completion --shell fish | source' >> ~/.config/fish/config.fish | |||
pixi completion --shell > ~/.config/fish/completions/pixi.fish |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just one thing to note - this changes the completions to be "non-dynamic", right?
ie. someoen might have to update their completions file after a pixi release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm you are right. Let me test something.
For context, this is what starship does, where I also copied nushell from: https://starship.rs/#fish |
Sure, but I don't really like cluttering the main fish script, thats what the completions or even the conf directory is for. |
I trust you on that, just wanted to add a data point :) |
Output to the
completions
directory, this seems more of the fishy way, AFAIK.