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.

Sign up for GitHub

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

Jump to bottom

Treeshaking introduces stray semi colons which cause the code to break #3941

Closed

ashubham opened this issue Jan 29, 2021 · 2 comments · Fixed by #3943

ashubham commented Jan 29, 2021

Rollup Version: 2.38.1
Operating System (or Browser): OSX
Node Version (if applicable): 12.10
Link to reproduction (IMPORTANT, read below): https://repl.it/@ashubham/rollup-stray-semicolon-repro#rollup.config.js

Expected Behavior

Build should succeed.

Actual Behavior

SyntaxError: Unexpected token punc «;», expected punc «)»

If you notice in the repl, if you turn treeshake: false the error goes away and the build is successful.

The text was updated successfully, but these errors were encountered:

Member

lukastaegert commented Jan 29, 2021

Thanks, minimal reproduction: https://rollupjs.org/repl/?version=2.38.1&shareable=JTdCJTIybW9kdWxlcyUyMiUzQSU1QiU3QiUyMm5hbWUlMjIlM0ElMjJtYWluLmpzJTIyJTJDJTIyY29kZSUyMiUzQSUyMmZvcih2YXIlMjBhJTIwJTNEJTIwZ2xvYmFsRnVuY3Rpb24oKSUyQyUyMGIlMjAlM0QlMjAwJTNCJTIwYiUyQiUyQiUzQiUyMGIlMjAlM0MlMjA1KSUyMCU3QiU1Q24lMjAlMjBjb25zb2xlLmxvZyhiKSUzQiU1Q24lN0QlMjIlMkMlMjJpc0VudHJ5JTIyJTNBdHJ1ZSU3RCU1RCUyQyUyMm9wdGlvbnMlMjIlM0ElN0IlMjJmb3JtYXQlMjIlM0ElMjJlcyUyMiUyQyUyMm5hbWUlMjIlM0ElMjJteUJ1bmRsZSUyMiUyQyUyMmFtZCUyMiUzQSU3QiUyMmlkJTIyJTNBJTIyJTIyJTdEJTJDJTIyZ2xvYmFscyUyMiUzQSU3QiU3RCU3RCUyQyUyMmV4YW1wbGUlMjIlM0FudWxsJTdE

lukastaegert mentioned this issue

Do not partially tree-shake unused declarations in for loops #3943

Merged

9 tasks

Member

lukastaegert commented Jan 31, 2021

Fix at #3943

lukastaegert closed this as completed in #3943

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment