Skip to content

Commit 58e223c

Browse files
committed
[DREAM-713] Bump @rollup/plugin-typescript to v12
Bumps @rollup/plugin-typescript ^8.3.3 -> ^12.3.0 so the rollup build stops emitting the TS5107 node10 deprecation warning under TypeScript 6.0. v12 validates that declarationDir sits alongside the bundle output. Declarations are already produced by the separate `tsc` step in script/build-assets, so declaration emit is disabled in rollup.config.js to satisfy the check and avoid redundant work. Ref: https://community.openproject.org/wp/DREAM-713
1 parent 37a7868 commit 58e223c

3 files changed

Lines changed: 48 additions & 61 deletions

File tree

package-lock.json

Lines changed: 43 additions & 59 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
"@primer/primitives": "^11.9.0",
7373
"@primer/stylelint-config": "^13.6.0",
7474
"@rollup/plugin-node-resolve": "^16.0.1",
75-
"@rollup/plugin-typescript": "^8.3.3",
75+
"@rollup/plugin-typescript": "^12.3.0",
7676
"axe-core": "^4.7.1",
7777
"chokidar-cli": "^3.0.0",
7878
"cssnano": "^7.0.1",

rollup.config.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,10 @@ export default {
1212
},
1313
plugins: [
1414
resolve(),
15-
typescript(),
15+
// Declarations are emitted by the separate `tsc` step (see script/build-assets),
16+
// so disable them here. Avoids @rollup/plugin-typescript v12 requiring
17+
// declarationDir to sit alongside the bundle output.
18+
typescript({declaration: false, declarationDir: undefined}),
1619
terser({keep_classnames: /Element$/}) // comment out terser in dev if you want debugger statements
1720
],
1821
onwarn: (warning, warn) => {

0 commit comments

Comments
 (0)