-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
fix(types): Fix issues in exported types #8333
fix(types): Fix issues in exported types #8333
Conversation
Codecov Report
@@ Coverage Diff @@
## main #8333 +/- ##
==========================================
+ Coverage 82.68% 82.69% +0.01%
==========================================
Files 113 113
Lines 7561 7578 +17
Branches 1818 1820 +2
==========================================
+ Hits 6252 6267 +15
- Misses 1309 1311 +2
... and 3 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
Looks good, thanks
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.
I don't think we can merge this because it adds a lot of unnecessary data to the dist
file(s).
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.
I think this is OK now, the unnecessary junk from package.json isn't being added. With this and #8218 we should be in a slightly better place wrt types.
Is it possible to merge this soon? |
Please merge this soon! |
I'd also appreciate if this would be merged soon 🙏 Currently, we have to do a bunch of @ts-ignore for types we know are correct. Thank you 🙏 |
@boris-petrov |
Description
Fixes #8301
Specific Changes proposed
Fix exported types.
Requirements Checklist