Use "builtin cd" instead of just "cd" #92
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Similar to #68 In case someone has overloaded the "cd" command (with alias), the plugin's behaviour can be odd. E.g. if a cd alias also types in "ls" after "cd", the autocompletion will spew out directory contents each time user hits tab.
|
Does this work for the supported shells? |
|
What are the supported shells? Given the name of the project, file and so, I would assume it is "bash".
By the way my PR#68 should have used the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Similar to #68
In case someone has overloaded the "cd" command (with alias), the
plugin's behaviour can be odd. E.g. if a cd alias also types in "ls"
after "cd", the autocompletion will spew out directory contents each
time user hits tab.
This MR amends said problem by ensuring cd is default version.