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

Provide plugin typings #453

Merged
merged 6 commits into from
Sep 30, 2017
Merged

Provide plugin typings #453

merged 6 commits into from
Sep 30, 2017

Conversation

XVincentX
Copy link
Member

@XVincentX XVincentX commented Sep 30, 2017

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.

@codecov
Copy link

codecov bot commented Sep 30, 2017

Codecov Report

Merging #453 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c0307f9...254e862. Read the comment docs.

Copy link
Contributor

@DrMegavolt DrMegavolt left a 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

@XVincentX
Copy link
Member Author

Yeah - I was thinking about the JWT one

@DrMegavolt
Copy link
Contributor

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

@XVincentX
Copy link
Member Author

All right - it seems reasonable.

@XVincentX XVincentX merged commit c49413e into master Sep 30, 2017
@XVincentX XVincentX deleted the features/plugin-typings branch September 30, 2017 21:36
gatherchou pushed a commit to yilu-tech/express-gateway that referenced this pull request Jul 29, 2021
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.

Provide a Typescript interface for plugin framework
2 participants