-
Notifications
You must be signed in to change notification settings - Fork 350
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
Provide plugin typings #453
Conversation
Codecov Report
@@ Coverage Diff @@
## master #453 +/- ##
=======================================
Coverage 90.89% 90.89%
=======================================
Files 226 226
Lines 9271 9271
=======================================
Hits 8427 8427
Misses 844 844 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, as next step we can do some example plugin written in ts
Yeah - I was thinking about the JWT one |
I don't agree with JWT, because this is extremely popular and will be forked a lot. So having it TS is a barrier for that. There are some smaller examples like web sockets, advanced logger (with connectors to ELK etc.). They can be TS no problem, as well as ES6 style with import\export |
All right - it seems reasonable. |
…ugin-typings Provide plugin typings
Connects #433 closes #433
This pull request provides basic Typings for the exposed plugin interface. People working with Typescript or even Javascript will receive a great help while writing code.