Skip to content

fix: quote -- for *Nushell* #1070

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

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

Conversation

stevenxxiu
Copy link

Fixes #1067.

@rawkode
Copy link

rawkode commented May 28, 2025

Hit this bug too. Thanks for working it out and submitting a PR!

@ajeetdsouza
Copy link
Owner

@stevenxxiu while I can merge the PR itself, this does seem like a bug in Nushell that the team should be made aware of. Would you be able to create an issue on their tracker with a minimum reproducible example of what is failing? I suspect on your shell, something like this should be failing as well:

zoxide add -- .

I would report the bug myself, but I'm somehow still not able to reproduce the error.

@otakutyrant
Copy link

I encountered the bug and the result of zoxide add -- . is zoxide: not a directory: /home/otakutyrant/Projects/stanza/-- here.

@stevenxxiu
Copy link
Author

stevenxxiu commented May 29, 2025

From the Nushell issue I reported, I found that this is caused by https://github.com/ajeetdsouza/zoxide/blob/v0.9.8/contrib/completions/zoxide.nu#L10. Due to the definition of zoxide add there, -- is treated as a path.

I'm not sure if this is something Nushell can fix easily then. @ajeetdsouza how do you think this should be handled?

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.

[regression] zoxide is broken in nushell
4 participants