[TASK] Unify Rollup usage in frontend tooling
In the current frontend tooling, we invoke Rollup twice: * via grunt-rollup to build library modules * via the command line to build everything related to CKEditor This commit gets rid of the first way and moves the library compilation to a JS-backed configuration as well, so we can get rid of a grunt package and also unify the tooling a bit. We also make use of @rollup/plugin-terser instead of the deprecated rollup-plugin-terser module that was used in ckeditor build before. To match the new behaviour, the "exec:ckeditor" task also was renamed to "exec:rollup". Executed commands: npm remove grunt-rollup npm remove rollup-plugin-terser npm install @rollup/plugin-terser --save-dev npm run build Resolves: #103686 Releases: main, 12.4 Change-Id: I8057dcceae2d039c637c6db68cf51b3ef1f12406 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83942 Tested-by:Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de>
Showing
- Build/Gruntfile.js 2 additions, 123 deletionsBuild/Gruntfile.js
- Build/package-lock.json 32 additions, 46 deletionsBuild/package-lock.json
- Build/package.json 1 addition, 2 deletionsBuild/package.json
- Build/rollup.config.js 13 additions, 0 deletionsBuild/rollup.config.js
- Build/rollup/bootstrap.js 21 additions, 0 deletionsBuild/rollup/bootstrap.js
- Build/rollup/chartjs.js 24 additions, 0 deletionsBuild/rollup/chartjs.js
- Build/rollup/ckeditor.js 6 additions, 6 deletionsBuild/rollup/ckeditor.js
- Build/rollup/ckeditor/build-translations-config.js 0 additions, 0 deletionsBuild/rollup/ckeditor/build-translations-config.js
- Build/rollup/lodash.js 22 additions, 0 deletionsBuild/rollup/lodash.js
- Build/rollup/select-pure.js 31 additions, 0 deletionsBuild/rollup/select-pure.js
- Build/rollup/shim/style-inject.js 0 additions, 0 deletionsBuild/rollup/shim/style-inject.js
- typo3/sysext/backend/Resources/Public/JavaScript/Contrib/lodash-es.js 3 additions, 2 deletions.../backend/Resources/Public/JavaScript/Contrib/lodash-es.js
- typo3/sysext/backend/Resources/Public/JavaScript/Contrib/select-pure.js 12 additions, 12 deletions...ackend/Resources/Public/JavaScript/Contrib/select-pure.js
- typo3/sysext/core/Resources/Public/JavaScript/Contrib/bootstrap.js 2 additions, 2 deletions...ext/core/Resources/Public/JavaScript/Contrib/bootstrap.js
- typo3/sysext/dashboard/Resources/Public/JavaScript/Contrib/chartjs.js 3 additions, 3 deletions.../dashboard/Resources/Public/JavaScript/Contrib/chartjs.js
- typo3/sysext/rte_ckeditor/Resources/Public/Contrib/@ckeditor/ckeditor5-ui.js 1 addition, 1 deletion...editor/Resources/Public/Contrib/@ckeditor/ckeditor5-ui.js
- typo3/sysext/rte_ckeditor/Resources/Public/Contrib/@ckeditor/ckeditor5-utils.js 1 addition, 1 deletion...tor/Resources/Public/Contrib/@ckeditor/ckeditor5-utils.js
Please register or sign in to comment