Replies: 1 comment 1 reply
-
|
I have no interest in converting this package to TypeScript, and would not merge any PR that did so. PRs that enhance documentation would be welcome, however. |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
Thanks for the feedback! I'll start working on some documentation improvements. |
Beta Was this translation helpful? Give feedback.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Convert Passport.js Core Codebase to TypeScript
Passport.js is a great library for all the underpinnings of authentication, especially for Express apps.
But it could be better!
One way it could be improved is with better type-checking and documentation.
I would be willing to make the PR for a complete migration from plain JS to TypeScript for this repo and any other core libs that I have the time for.
And for documentation I would need someone more experienced with this repo and the Passport.js ecosystem for some help on intended usage.
Beta Was this translation helpful? Give feedback.
All reactions