-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[WIP] Bring typings from DefinitelyTyped into this repo #690
[WIP] Bring typings from DefinitelyTyped into this repo #690
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit b0d9a33:
|
@@ -0,0 +1,59 @@ | |||
// TypeScript Version: 3.8 |
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.
Bumped this from 3.1 to 3.8 for this support:
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-8.html#type-only-imports-and-exports
Looks awesome. I don't know about the dtslint stuff, but once that's worked out I'm good to merge 👍 Thanks! |
Add `pure.d.ts` that re-exports everything from `index.d.ts` Turn off `whitespace` rule
Codecov Report
@@ Coverage Diff @@
## master #690 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 4 4
Lines 99 99
Branches 15 15
=========================================
Hits 99 99 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.
Sweet! Thanks!
@all-contributors please add @samtsai for code and tests |
I've put up a pull request to add @samtsai! 🎉 |
🎉 This PR is included in version 10.1.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Thanks so much for your help! I've added you as a collaborator on the project. Please make sure that you review the |
What:
Improve support of typings by bringing them into this repo instead of relying on a versioned external repo
Why:
Help with stability for consumers using typings and give a clearer understanding of what should be supported in a given version without having to reference multiple repos.
How:
Modeled after these changes in a sibling repo: testing-library/dom-testing-library#530
Checklist:
docs site
Currently this does not validate with several errors for Typescript@4.0, example:
I'm still investigating this, but if anyone knows more about
dtslint
I'm all 👂