-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
Correctly handling __proto__ export as module object key #5313
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5313 +/- ##
==========================================
- Coverage 98.82% 98.81% -0.02%
==========================================
Files 232 232
Lines 8973 8981 +8
Branches 2338 2341 +3
==========================================
+ Hits 8868 8875 +7
Misses 46 46
- Partials 59 60 +1 ☔ View full report in Codecov by Sentry. |
@lukastaegert ready to merge. sorry for wasting too much time to deal with existing tests with hash... |
BTW you can use |
@lukastaegert Under your patient guidance, the heavy maintenance work suddenly became enjoyable! |
This PR has been released as part of rollup@4.9.3. You can test it via |
4.9.3 introduced a regression in rollup/rollup#5313 which changes the `export * from` codegen and breaks Node.js' re-export detection.
4.9.3 introduced a regression in rollup/rollup#5313 which changes the `export * from` codegen and breaks Node.js' re-export detection.
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers:
Description
https://rollupjs.org/repl/?version=4.9.1&shareable=JTdCJTIyZXhhbXBsZSUyMiUzQW51bGwlMkMlMjJtb2R1bGVzJTIyJTNBJTVCJTdCJTIyY29kZSUyMiUzQSUyMmltcG9ydCUyMColMjBhcyUyMHglMjBmcm9tJTIwJy4lMkZ4JyUzQiU1Q25jb25zb2xlLmxvZyh4KSUzQiUyMiUyQyUyMmlzRW50cnklMjIlM0F0cnVlJTJDJTIybmFtZSUyMiUzQSUyMm1haW4uanMlMjIlN0QlMkMlN0IlMjJjb2RlJTIyJTNBJTIyZXhwb3J0JTIwY29uc3QlMjBfX3Byb3RvX18lMjAlM0QlMjAwJTNCJTIyJTJDJTIyaXNFbnRyeSUyMiUzQWZhbHNlJTJDJTIybmFtZSUyMiUzQSUyMnguanMlMjIlN0QlNUQlMkMlMjJvcHRpb25zJTIyJTNBJTdCJTIyb3V0cHV0JTIyJTNBJTdCJTIyZm9ybWF0JTIyJTNBJTIyZXMlMjIlN0QlMkMlMjJ0cmVlc2hha2UlMjIlM0F0cnVlJTdEJTdE
The PR generate
["__proto__"]:
instead of__proto__:
. It can also be written byget __proto__ ()
instead, if that would be better. All formats have no__proto__
related problem now.