Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Upgrade fuzzaldrin #3443

Merged
merged 2 commits into from
Sep 4, 2014
Merged

Upgrade fuzzaldrin #3443

merged 2 commits into from
Sep 4, 2014

Conversation

kevinsawicki
Copy link
Contributor

This includes a fix on Windows so that the library that scores matches displayed in the fuzzy finder (and elsewhere) uses the right native path separator so it should now display the same results on all platforms.

This also includes many other fuzzy scoring fixes that have come in the last couple months: atom/fuzzaldrin@v1.1.0...v2.1.0

The results displayed in the fuzzy finder may have changed due to fixes but hopefully all the changes are an improvement and the best matches are still shown first.

If anyone has a chance to try this out (on Windows, and other platforms), that would be great.

Please comment here if you see any weird ordering or slowness when using the fuzzy finder, command palette, etc.

@kevinsawicki
Copy link
Contributor Author

Screenshots of the issue being addressed by this upgrade available on atom/fuzzaldrin#11

@kevinsawicki kevinsawicki force-pushed the ks-upgrade-fuzzaldrin branch 2 times, most recently from ee63bfd to 6416983 Compare September 4, 2014 16:02
kevinsawicki added a commit that referenced this pull request Sep 4, 2014
@kevinsawicki kevinsawicki merged commit d8be038 into master Sep 4, 2014
@kevinsawicki kevinsawicki deleted the ks-upgrade-fuzzaldrin branch September 4, 2014 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant