autojump icon indicating copy to clipboard operation
autojump copied to clipboard

Fix duplicate matches

Open rico-chet opened this issue 3 years ago • 3 comments

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

rico-chet avatar Jan 24 '22 11:01 rico-chet

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

CharlesARoy avatar Jan 30 '22 01:01 CharlesARoy

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 avatar Feb 08 '22 20:02 rico-chet

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.

rico-chet avatar Mar 30 '22 11:03 rico-chet