Skip to content

Conversation

@Tom910
Copy link

@Tom910 Tom910 commented Mar 13, 2019

Added parameter sideEffects: false in packages.json, because webpack used this parameters for effective tree shaking - https://webpack.js.org/guides/tree-shaking/#mark-the-file-as-side-effect-free

@theKashey
Copy link

Corresponding webpack issue - microsoft/TypeScript#16999
Corresponding rollup issue - rollup/rollup-plugin-typescript#139

@DanielRosenwasser
Copy link
Member

Seems like this has already been done in #72.

@Tom910 Tom910 deleted the np-side-effects branch May 7, 2020 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants