We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
run.target: ["chromium"]
bug
A plain $ web-ext run will read/validate webExt.run.target from my package.json, but then it ignores it.
$ web-ext run
webExt.run.target
// package.json { "webExt": { "run": { "target": ["chromium"] } } }
This verbose log line explains why: the CLI parser is providing its own defaults before merging with the config file.
[web-ext/lib/config.js][debug] Favoring CLI: target=firefox-desktop over configuration: target=chromium
The config file should be respected by applying the defaults after reading the config file.
node --version && npm --version && web-ext --version v23.1.0 10.9.0 8.3.0
The text was updated successfully, but these errors were encountered:
This message is emitted from
web-ext/src/config.js
Lines 121 to 124 in 958e1d4
Sorry, something went wrong.
the CLI parser is providing its own defaults before merging with the config file.
Likely here:
web-ext/src/program.js
Line 583 in 64ad32f
Yargs is passing the default the same way it would pass a user-provided flag, therefore overriding any config that is found later.
I'd review other such defaults in the file. If they're all ignored the same way, they should probably all be removed.
Yargs' own docs/bugtracker probably detail how to deal with configuration vs flag overrides.
No branches or pull requests
Is this a feature request or a bug?
bug
What is the current behavior?
A plain
$ web-ext run
will read/validatewebExt.run.target
from my package.json, but then it ignores it.This verbose log line explains why: the CLI parser is providing its own defaults before merging with the config file.
What is the expected or desired behavior?
The config file should be respected by applying the defaults after reading the config file.
Version information (for bug reports)
The text was updated successfully, but these errors were encountered: