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

add trailing else block to catch everything else #113

Merged
merged 1 commit into from
Jun 22, 2024

Conversation

tomeichlersmith
Copy link
Owner

I've been annoyed when I notice I can't tab complete path names in positions after the second position and so adding this final else block restore default bash completion for any situation that isn't specific to denv.

@tomeichlersmith
Copy link
Owner Author

I was able to do a simple confirmation that this extra else block works as long as the completions are re-initialized (new terminal or re-sourcing of completion attaching script).

tom@nixos:~/code/denv$ git branch
  102-uid-on-macos
  default-comp-by-default
* main
tom@nixos:~/code/denv$ . completions/denv 
tom@nixos:~/code/denv$ denv bash man/ ma # pressing Tab has no response^C
tom@nixos:~/code/denv$ git switch default-comp-by-default 
Switched to branch 'default-comp-by-default'
Your branch is up to date with 'origin/default-comp-by-default'.
tom@nixos:~/code/denv$ . completions/denv 
tom@nixos:~/code/denv$ denv bash man/ man/ # pressing Tab completes directory like first argument

@tomeichlersmith tomeichlersmith merged commit 39040dd into main Jun 22, 2024
9 checks passed
@tomeichlersmith tomeichlersmith deleted the default-comp-by-default branch June 22, 2024 17:11
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