Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement optional fuzzy search #11

Merged
merged 4 commits into from Sep 30, 2016
Merged

Conversation

trading-peter
Copy link
Contributor

Using the super fast fuzzysearch module and a new option we can now perform a fuzzy search for emojis. This is espicaly usefull if one don't know the english spelling of some keywords that well.

Using the super fast fuzzysearch module and a new option we can now perform a fuzzy search for emojis. This is espicaly usefull if one don't know the english spelling of some keywords that well.
@watilde
Copy link
Owner

watilde commented Sep 29, 2016

Wooo this feature looks nice! Could you take a look at the error on travis CI?
https://travis-ci.org/watilde/emoji-cli/builds/163655669

It would be great if you could fix it 😺

@trading-peter
Copy link
Contributor Author

Should be all good now. Missed a space...

@watilde
Copy link
Owner

watilde commented Sep 29, 2016

A w e s o m e !

I will add a test case for the fuzzy search later(maybe this weekend), or you could add it if you have an interest in it 😼

Peter Kaske added 2 commits September 30, 2016 07:47
Before, only keywords where fuzzy searched.
@trading-peter
Copy link
Contributor Author

I added some tests. While writing them I realized that fuzzy search didn't search in the emoji name, only the keywords. I fixed that too.

@watilde
Copy link
Owner

watilde commented Sep 30, 2016

Great!! That's the benefit of the test we can receive. Let me merge this and ship it immediately 😆

Thanks for putting this together and following up hardly

@watilde watilde merged commit 3fd4928 into watilde:master Sep 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants