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
feat: deduplicate reexports and renderedExports to simplify output #4759
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4759 +/- ##
==========================================
- Coverage 99.04% 99.03% -0.02%
==========================================
Files 217 217
Lines 7734 7749 +15
Branches 2143 2147 +4
==========================================
+ Hits 7660 7674 +14
Misses 24 24
- Partials 50 51 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
I think the current changes are generating wrong |
Yes, then do it just for |
This PR has been released as part of rollup@3.8.0. You can test it via |
This PR contains:
Are tests included?
I think the existing test changes should cover it, but I'm not sure if the modifications of the 'system' test are correct, I need to learn about SystemJS.
Breaking Changes?
List any relevant issue numbers:
resolves #4698
Description