ChrisTitusTech / powershell-profile

Pretty PowerShell that looks good and functions almost as good as Linux terminal
504 stars 293 forks source link

Legacy support for zoxide #87

Open korneltlaczala opened 1 week ago

korneltlaczala commented 1 week ago

Problem

As some of us have noticed, after recent changes (pull request https://github.com/ChrisTitusTech/powershell-profile/pull/70), z and zi commands are no longer supported. I described the issue more extensively in https://github.com/ChrisTitusTech/powershell-profile/issues/86#issuecomment-2186920624

In short:

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 https://github.com/ChrisTitusTech/powershell-profile/issues/86#issuecomment-2186920624)

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 https://github.com/ChrisTitusTech/powershell-profile/pull/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 commented 1 week ago

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