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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
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