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

Use different zxcvbn fork #36

Closed
mrienstra opened this issue May 4, 2021 · 2 comments
Closed

Use different zxcvbn fork #36

mrienstra opened this issue May 4, 2021 · 2 comments

Comments

@mrienstra
Copy link

Continuation of #24:

npmjs.com/zxcvbn3 doesn't appear to be actively maintained, it currently points to github.com/hrueger/zxcvbn which is "1 commit ahead, 59 commits behind zxcvbn-ts:master". That single commit (diff) is adding this line to README.md:

⚠ Moved to https://github.com/MrWook/zxcvbn

github.com/MrWook/zxcvbn in turn, is "1 commit ahead of dropbox:master", and that single commit (diff) is modifying README.md to link to github.com/zxcvbn-ts/zxcvbn.

... Which looks considerably more "alive" (here's snyk.io's take).

NPM: https://www.npmjs.com/package/@zxcvbn-ts/core
Migration guide: https://zxcvbn-ts.github.io/zxcvbn/guide/migration/
Repo: https://github.com/zxcvbn-ts/zxcvbn

PS: The creators of those first two forks (hrueger, MrWook) have also contributed to zxcvbn-ts/zxcvbn, as per this comment.
PPS: Opened PRs to clarify the situation: 1, 2.

@40rn05lyv
Copy link

Totally agree, please update the README file which is misleading.

@antoantonyk
Copy link
Owner

@mrienstra I have integrated the zxcvbn-ts.

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

No branches or pull requests

3 participants