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

drop support for python 2 and add some typing #62

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

fuhrysteve
Copy link
Contributor

@fuhrysteve fuhrysteve commented Feb 4, 2021

Not sure if you're open to dropping python 2 support, but I thought I'd test the waters!

Nothing too exciting, just adding typing on the user-facing zxcvbn() function and in a couple other super easy places, if only to make sure mypy & github actions are actually working.

Please note that this fully breaks backward compatibility with python 3.5 or lower, as it uses features introduced in python 3.6.

@dwolfhub
Copy link
Owner

dwolfhub commented Feb 5, 2021

Given it's been EOL for a year now it's probably time. I would just want to bump a major version.

@fuhrysteve
Copy link
Contributor Author

Bumped the major version number to 5 and updated the docs!

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