Ensure we can cd into directories named "-"#626
Ensure we can cd into directories named "-"#626twe4ked wants to merge 2 commits intonushell:masterfrom
Conversation
3966332 to
465cb57
Compare
|
@twe4ked Investigated usage of All cases are already covered before this PR. Are we missing something? |
|
The new case that I added. The implementation on Copy the new test case into |
|
It's this case from the original test. It wasn't very clear because it was all one big test. |
|
@twe4ked This is what I'm assuming default behavior should be, meaning that if in your working directory there is a directory named What behavior should we expect from this? @jonathandturner @wycats Assuming there is a directory named
|
|
Oh interesting. I assumed the default behaviour would be to change to the directory. Just confirmed that |
|
Still, sometimes the legacy behaviors is something we don't want. |
|
I would say |
|
👍 I'm cool with that. I'll close, we can always re-open if people feel differently. |
|
@twe4ked we will probably take things back to the drawing board (as suggested by @wycats) since |
I suggest reviewing the individual commits. The actual fix is quite small but I then extracted
cd_path()to try make the error returning simpler. Feel free to cherry-pick the actual functionality without the function extraction.