Skip to content

Commit

Permalink
feat: mark packages as sideEffects: false
Browse files Browse the repository at this point in the history
reverts #2192 to remove only sideEffect in engine-core package
  • Loading branch information
AviVahl committed Dec 15, 2024
1 parent 77df074 commit 52ac226
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 19 deletions.
3 changes: 2 additions & 1 deletion packages/core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,5 +30,6 @@
"license": "MIT",
"publishConfig": {
"access": "public"
}
},
"sideEffects": false
}
13 changes: 0 additions & 13 deletions packages/core/src/polyfills/report-error-polyfill.ts

This file was deleted.

2 changes: 0 additions & 2 deletions packages/core/src/runtime-main.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import './polyfills/report-error-polyfill.js';

import type { IRunOptions, TopLevelConfig } from './types.js';
import type { AnyEnvironment } from './entities/index.js';
import { FeatureLoadersRegistry, IFeatureLoader } from './run-engine-app.js';
Expand Down
3 changes: 2 additions & 1 deletion packages/engine-cli/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -41,5 +41,6 @@
"license": "MIT",
"publishConfig": {
"access": "public"
}
},
"sideEffects": false
}
3 changes: 2 additions & 1 deletion packages/runtime-node/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,5 +32,6 @@
"license": "MIT",
"publishConfig": {
"access": "public"
}
},
"sideEffects": false
}
3 changes: 2 additions & 1 deletion packages/test-kit/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,5 +37,6 @@
"license": "MIT",
"publishConfig": {
"access": "public"
}
},
"sideEffects": false
}

0 comments on commit 52ac226

Please sign in to comment.