Skip to content

Commit

Permalink
fix: Use webpack -o instead of --output
Browse files Browse the repository at this point in the history
Currently the build/test fails with the following output:

```
Test webextension-polyfill bundled with webpack
===============================================
[webpack-cli] Unknown argument: --output
[webpack-cli] Unknown argument: /tmp/webpack-bundle.js
? Which flags do you want to use? …
✔ --entry: The entry point(s) of your application e.g. ./src/main.js
✔ --config: Provide path to a webpack configuration file e.g. ./webpack.config.js
✔ --color: Enable/Disable colors on console
✔ --merge: Merge two or more configurations using webpack-merge e.g. -c ./webpack.config.js -c ./webpack.test.config.js --merge
✔ --progress: Print compilation progress during build
✔ --help: Outputs list of supported flags
✔ --output-path: Output location of the file generated by webpack e.g. ./dist/
```

Regressed by:
webpack/webpack-cli#1855
webpack/webpack-cli@2f36b9d

The `-o` flag is compatible with both the old and new flag name.
  • Loading branch information
Rob--W committed Oct 14, 2020
1 parent 01fb67f commit 14ccca5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scripts/run-module-bundlers-smoketests.js
Expand Up @@ -8,7 +8,7 @@ Test webextension-polyfill bundled with webpack
===============================================`);

process.env.TEST_BUNDLED_POLYFILL = "/tmp/webpack-bundle.js";
result = shell.exec(`webpack --mode production --entry ./test/fixtures/bundle-entrypoint.js --output ${process.env.TEST_BUNDLED_POLYFILL}`).code ||
result = shell.exec(`webpack --mode production --entry ./test/fixtures/bundle-entrypoint.js -o ${process.env.TEST_BUNDLED_POLYFILL}`).code ||
shell.exec("npm run test").code || result;

console.log(`
Expand Down

0 comments on commit 14ccca5

Please sign in to comment.