Skip to content
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

complete/fish: Take RIPGREP_CONFIG_PATH into account #2708

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blyxxyz
Copy link
Contributor

@blyxxyz blyxxyz commented Jan 10, 2024

The fish completions now also pay attention to the configuration file to determine whether to suggest negation options and not just to the current command line.

This doesn't cover all edge cases. For example the config file is cached, and so changes may not take effect until the next shell session. But the cases it doesn't cover are hopefully very rare.

See the post hoc discussion on #2684.

cc @cstyles

The fish completions now also pay attention to the configuration file
to determine whether to suggest negation options and not just to the
current command line.

This doesn't cover all edge cases. For example the config file is
cached, and so changes may not take effect until the next shell
session. But the cases it doesn't cover are hopefully very rare.
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.

1 participant