You Should Use reminder fires even when I use the correct alias about zsh-you-should-use

[MichaelAquilina]

Hi @jcklpe. This is a known issue in the 1.5.0 release. Please update to the latest version and the problem should go away

[MichaelAquilina]

Going to close this issue. Let me know if you still see this problem on the latest version!

[jcklpe]

@MichaelAquilina

┌─    /mnt/c/Users/aslan/home/Documents/configs/zsh/zsh-plugins/zsh-you-should-use                                         master   
└─➤ git status 
On branch master 
Your branch is up to date with 'origin/master'.

nothing to commit, working tree clean

┌─    /mnt/c/Users/aslan/home/Documents                                                                                                
└─➤ echo "$YSU_VERSION" 
1.5.2 

I've updated it and I'm still getting the issue.

[MichaelAquilina]

That's strange - I can't reproduce the issue myself. I'll dig into your zshrc and see if there's anything specific in there that might be causing this.

[jcklpe]

I think it might only be a problem for commands that I have multiple aliases for

┌─    /mnt/c/Users/aslan/home/Documents/configs/zsh                                                                      master    
└─➤ .. 
Found existing alias for "cd ..". You should use: "cd.." 
android-debug-bridge  bash     conky    git  hyper-js          js-beautify  npm     README.md  ueli         vscode   wordpress  zsh 
auto-hotkey           con-emu  firefox  hue  install-list.txt  LICENSE      ranger  tour-box   user-styles  windows  zBrush     

┌─    /mnt/c/Users/aslan/home/Documents/configs                                                                          master    
└─➤ cd..   
Found existing alias for "cd ..". You should use: ".." 
3Dpainting.pdf  configs      health-experiments.md  movie-list.txt  parents.jpg  travel-list.txt 
Adobe           Desktop.ini  journal                paperwork       themes       writing

[MichaelAquilina]

ah yep I just found that regression this morning. I have a failing test case in #89. If I cant get that out by tonight I'll provide a hotfix.

A new feature has ended up introducing these regressions and tests did not catch them unfortunately.

[jcklpe]

Cool, thanks!

[MichaelAquilina]

@jcklpe please check out the latest release.

I've reverted the feature that introduced this bug. I'll look into introducing it again correctly at another time.

[jcklpe]

Yup works like previous now. Thanks for making this, it's a super great utility :)