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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

include types with package #235

Open
jamiehaywood opened this issue May 16, 2021 · 5 comments
Open

include types with package #235

jamiehaywood opened this issue May 16, 2021 · 5 comments
Labels

Comments

@jamiehaywood
Copy link

jamiehaywood commented May 16, 2021

Would you consider including the types as part of the package? I appreciate that you can get them from the @types/cors package, but it would be a nicer dev xp to have them included 馃檪

@dougwilson
Copy link
Contributor

As long as we have a volunteer that understands typescript to be available to hrlp with any issues, review changes to ensure the types are updated correctly, etc. This is important as with the separate package an issue with the types can be fixed independently of the package; including them in the package means they need to be accurate with every publish of the package, so need someone who understands typings to be on call for any changes that need to land.

@jamiehaywood
Copy link
Author

I would be happy to volunteer. I use TypeScript extensively at work and in personal projects.

@dougwilson
Copy link
Contributor

Awesome @jamiehaywood ! I would think probably the best way to get started would be to help get closed the open issues over at https://github.com/DefinitelyTyped/DefinitelyTyped/issues so then we have the @types/cors at what would get included here without any open issues from folks. Then we can copy them into this repo and go forward. Does that sound like a good plan? If not, let me know what you think the plan should be 馃憤

@jamiehaywood
Copy link
Author

Yep that plan sounds good - I will take a look over the next couple of the days & report back!

@monsieurnebo

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants