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

Legacy support for zoxide #87

Open
korneltlaczala opened this issue Jun 24, 2024 · 1 comment
Open

Legacy support for zoxide #87

korneltlaczala opened this issue Jun 24, 2024 · 1 comment

Comments

@korneltlaczala
Copy link
Contributor

Problem

As some of us have noticed, after recent changes (pull request #70), z and zi commands are no longer supported.
I described the issue more extensively in #86 (comment)

In short:

  • use cd instead of z
  • use cdi instead of zi

Suggestion

Just to be clear. If you want to bring back the old functionality of z and zi, you can do so in your profile.ps1 file. (Look #86 (comment))

This change could be confusing for recurring users and for those that have become very used to the z, zi commands.

Maybe there is nothing wrong with leaving the support for z and zi natively, alongside the cd, cdi commands
I would actually think that was the intention of #70 (@TakenMC what do you think?)

Why should one even start using cd over z?

Even though it's longer, it is more versitile and supported everywhere. It requires less getting used to if one regularly uses VMs or SSH to log remotely. That´s why it may be better for some users, and thus both cd and z should be supported.

@bitsper2nd
Copy link

Why not make a Pull Request to bring back the old alias for the prompt and set new aliases?

Invoke-Expression (& { (zoxide init powershell | Out-String) })

Remove-Alias -Name "cd"
New-Alias -Name "cd" z
New-Alias -Name "cdi" zi

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

No branches or pull requests

2 participants