Skip to content

Commit

Permalink
chore: revert potentially unsafe treeshake rules
Browse files Browse the repository at this point in the history
  • Loading branch information
merceyz committed Dec 1, 2021
1 parent c00d29e commit c5d2f9d
Show file tree
Hide file tree
Showing 6 changed files with 14 additions and 23 deletions.
13 changes: 12 additions & 1 deletion .pnp.cjs

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 0 additions & 5 deletions packages/vscode-zipfs/rollup.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,6 @@ const mode = process.env.NODE_ENV || `production`;
// eslint-disable-next-line arca/no-default-export
export default async () =>
defineConfig({
treeshake: {
propertyReadSideEffects: false,
unknownGlobalSideEffects: false,
tryCatchDeoptimization: false,
},
input: `./sources/index.ts`,
output: {
file: `./build/index.js`,
Expand Down
5 changes: 0 additions & 5 deletions packages/yarnpkg-core/rollup.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,6 @@ function wrapOutput() {

// eslint-disable-next-line arca/no-default-export
export default defineConfig({
treeshake: {
propertyReadSideEffects: false,
unknownGlobalSideEffects: false,
tryCatchDeoptimization: false,
},
input: `./sources/worker-zip/Worker.ts`,
output: {
file: `./sources/worker-zip/index.js`,
Expand Down
2 changes: 1 addition & 1 deletion packages/yarnpkg-core/sources/worker-zip/index.js

Large diffs are not rendered by default.

10 changes: 0 additions & 10 deletions packages/yarnpkg-pnp/rollup.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,6 @@ function wrapOutput() {
// eslint-disable-next-line arca/no-default-export
export default defineConfig([
{
treeshake: {
propertyReadSideEffects: false,
unknownGlobalSideEffects: false,
tryCatchDeoptimization: false,
},
input: `./sources/loader/_entryPoint.ts`,
output: {
file: `./sources/hook.js`,
Expand Down Expand Up @@ -59,11 +54,6 @@ export default defineConfig([
],
},
{
treeshake: {
propertyReadSideEffects: false,
unknownGlobalSideEffects: false,
tryCatchDeoptimization: false,
},
input: `./sources/esm-loader/loader.ts`,
output: {
file: `./sources/esm-loader/built-loader.js`,
Expand Down
2 changes: 1 addition & 1 deletion packages/yarnpkg-pnp/sources/hook.js

Large diffs are not rendered by default.

0 comments on commit c5d2f9d

Please sign in to comment.