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
Do not count accessing members of an external namespace as side-effects. #4001
Conversation
Thank you for your contribution! ❤️You can try out this pull request locally by installing Rollup via
or load it into the REPL: |
Codecov Report
@@ Coverage Diff @@
## master #4001 +/- ##
=======================================
Coverage 97.24% 97.24%
=======================================
Files 191 191
Lines 6741 6742 +1
Branches 1967 1968 +1
=======================================
+ Hits 6555 6556 +1
Misses 99 99
Partials 87 87
Continue to review full report at Codecov.
|
@lukastaegert Thank you for this fix, it solved my issue! |
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers:
#3999
Description
This solves a regression introduced in #3999 where accessing properties of external namespaces was considered side-effectful, preventing tree-shaking.