-
Notifications
You must be signed in to change notification settings - Fork 90
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
Drop support for Jaro, Jaro-Winkler and Levenshtein distances #282
Drop support for Jaro, Jaro-Winkler and Levenshtein distances #282
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to deprecate the similarity argument
There are still place where python-levenshtein is remaining: I found one in .travis.yml, where I believe that it should be removed. Can you "git grep levenshtein", and check all the corresponding places that they do not need to be fixed. |
You're right, and that's somewhat purposeful, as Travis files will be removed by #287, as we don't use them anyway. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re-checked the code, looks good to me !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you
Resolves #226.