-
-
Notifications
You must be signed in to change notification settings - Fork 469
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
include types with package #235
Comments
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. |
I would be happy to volunteer. I use TypeScript extensively at work and in personal projects. |
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 |
Yep that plan sounds good - I will take a look over the next couple of the days & report back! |
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 🙂The text was updated successfully, but these errors were encountered: