From f927629c26ea2194c8c533fca7c5b06b3cc618df Mon Sep 17 00:00:00 2001 From: larabr <7375870+larabr@users.noreply.github.com> Date: Fri, 16 May 2025 11:38:34 +0200 Subject: [PATCH] Rollup: change config option capitalization --- .github/workflows/tests.yml | 4 ++-- package.json | 2 +- rollup.config.js | 12 ++++++------ 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index d2afd580..be4ca5a9 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -114,7 +114,7 @@ jobs: - name: Run browser tests (lightweight) # overwrite test/lib run: | - npm run build-test -- --configTestLightweightBuild + npm run build-test -- --config-test-lightweight-build npm run test-browser:ci -- --static-logging test-browsers-compatibility: @@ -162,7 +162,7 @@ jobs: - name: Run browserstack tests (lightweight) # overwrite test/lib run: | - npm run build-test -- --configTestLightweightBuild + npm run build-test -- --config-test-lightweight-build npm run test-browserstack -- --static-logging env: LIGHTWEIGHT: true diff --git a/package.json b/package.json index 0a42fb2a..b4c2eb5c 100644 --- a/package.json +++ b/package.json @@ -43,7 +43,7 @@ ], "scripts": { "build": "rollup --config", - "build-test": "npm run build -- --configBuildOnly=test", + "build-test": "npm run build -- --config-build-only=test", "prepare": "npm run build", "test": "mocha --timeout 120000 test/unittests.js", "test-type-definitions": "tsx test/typescript/definitions.ts", diff --git a/rollup.config.js b/rollup.config.js index b2c72fdf..20c4334f 100644 --- a/rollup.config.js +++ b/rollup.config.js @@ -172,20 +172,20 @@ const getBrowserTestBuild = useLightweightBuild => ({ * https://rollupjs.org/command-line-interface/#configuration-files * * The custom options we support are: - * @param commandLineArgs.configBuildOnly {'dist'|'node'|'lightweight'|'test'} to specify a build target; - * defaults to 'dist', which does not build tests. - * @param commandLineArgs.configTestLightweightBuild {Boolean}: in the context of building browser tests, - * whether the lightweight build should be included instead of the standard one + * - "config-build-only": 'dist'|'node'|'lightweight'|'test'|string - to specify a build target; + * defaults to 'dist', which does not build tests; + * - "config-test-lightweight-build": Boolean - in the context of building browser tests, + * whether the lightweight build should be included instead of the standard one */ export default commandLineArgs => Object.assign([ nodeBuild, fullBrowserBuild, lightweightBrowserBuild, - getBrowserTestBuild(commandLineArgs.configTestLightweightBuild) + getBrowserTestBuild(commandLineArgs['config-test-lightweight-build']) ].filter(rollupConfig => { rollupConfig.output = rollupConfig.output.filter(output => { return (output.file || output.dir + '/' + output.entryFileNames).includes( - commandLineArgs.configBuildOnly || 'dist' // Don't build test bundle by default. + commandLineArgs['config-build-only'] || 'dist' // Don't build test bundle by default. ); }); return rollupConfig.output.length;