-
Notifications
You must be signed in to change notification settings - Fork 23
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
Move to DT #19
Comments
I agree. |
Any movement on this? It'd be nice to be able to install this like more traditional type libraries; alternatively the types here could be added to the core mysql2 library via PR if you're alright with that. |
+1 |
1 similar comment
+1 |
@felixfbecker This is your decision 😄 It might make sense to authorize someone to copy the definition and hand off maintenance to DefinitelyTyped. For everyone here, note that a lot of people in @types spent a lot of time creating high quality definitions that ultimately were incompatible with the TypeScript operating model because we don't want to work on DefinitelyTyped (hard to iterate, hard to respond to issues, giant mono repo lacking individual ownership, etc) and this resulted in a lot of burn out (especially personally, I dropped OSS for the past year). |
@blakeembrey I feel you for sure on your issues with DT; that's why personally I prefer moving the definitions from here over into the actual mysql2 library. Assuming @sidorares is comfortable with that addition to the repository, I generally feel that including types with the actual package is a superior option to any external source of truth; see the neverending discussion around the Stripe library's types for an example of why. |
@moberemk @blakeembrey yes, happy to have type definitions in repo |
Best news I read this morning! Thanks @sidorares! |
Worth noting that, as things are now, that would mean including |
Any update? |
@Danger89 I haven't done anything myself. Happy to review PRs on this |
any updates on this one? would be very much appreciated! |
It sounds like we've got a green light from everybody involved, and are just waiting on someone to make a PR. Is this correct @sidorares @felixfbecker? Types for mysql2 will be needed on a large project that I'm helping modernize, so I can champion this. As ISC and MIT are largely compatible (IANAL but this is a widely held perspective), we can probably just embed this entire codebase in a |
You're welcome to do whatever with it. Pretty sure ISC and MIT are equivalent. |
@mike-marcacci yes, I'm just observer here, happy to merge types if quality looks good enough but I'm not the best person to author |
I think it would be a good idea to move this repo to DefinitelyTyped.
The text was updated successfully, but these errors were encountered: