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

Update python-levenshtein with levenshtein #61

Merged
merged 4 commits into from
Sep 2, 2022
Merged

Update python-levenshtein with levenshtein #61

merged 4 commits into from
Sep 2, 2022

Conversation

BramVanroy
Copy link
Contributor

python-levenshtein is not maintained and has been superseded by levenshtein. It includes the same functionality and also a C engine. Usage is the same. Updating to levenshtein should improve usability as it comes with prebuilt wheels and will continue to have support for at least the near future, which is good news for the fast iterations in Python development.

closes #60

Bram Vanroy added 2 commits September 2, 2022 14:49
python-levenshtein is not maintained. It lives on in levenshtein
@jitsi-jenkins
Copy link

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

@BramVanroy
Copy link
Contributor Author

Hi, thanks for your contribution! If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

Done.

@nikvaessen nikvaessen merged commit 0a0cb93 into jitsi:master Sep 2, 2022
@nikvaessen
Copy link
Collaborator

Bedankt voor de PR! :)

@BramVanroy
Copy link
Contributor Author

Thanks for the quick integration!

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.

Update Levenshtein dependency to maintained version
3 participants