search

wting / autojump

A cd command that learns - easily navigate directories from the command line
Other
16.06k stars 702 forks source link

Fix duplicate matches #646

Open rico-chet opened 2 years ago

rico-chet commented 2 years ago

Fixes #348, also addressing a couple warnings I stumbled upon.

CharlesARoy commented 2 years ago

@wting can you approve this PR? The duplicate results are the only thing keeping me from switching back from Zoxide.

rico-chet commented 2 years ago

The first version had an issue that the results weren't sorted and thus one would typically jump to a random directory instead of the chosen one. The new push fixes the issue by sorting the set.

rico-chet commented 2 years ago

autojump is a PITA to work on, for the test coverage is quite poor :-(. The change breaks jc functionality, please don't merge. Will have to add jc regression test first and fix it.