-
Notifications
You must be signed in to change notification settings - Fork 53
Open
Description
This project uses a fork of yargs with a config-booleans branch.
Line 32 in fa1b418
| "yargs": "git+https://github.com/sgentle/yargs.git#config-booleans" |
It looks like the new yargs-parser has changes related to default booleans, and this fork may no longer be required.
Snippet of some of the changes:
Object.keys(flags.configs).forEach(function (configKey) {
...
// setting arguments via CLI takes precedence over
// values within the config file.
if (argv[key] === undefined || (flags.defaulted[key])) {
delete argv[key]
setArg(key, config[key])
}
})
} catch (ex) {
if (argv[configKey]) error = Error(__('invalid json config file: %s', configPath))
}
}
})
}Snippet of some of current yargs-parser code:
Object.keys(config).forEach(function (key) {
...
// setting arguments via CLI takes precedence over
// values within the config file.
if (!hasKey(argv, fullKey.split('.')) || (flags.defaulted[fullKey]) || (flags.arrays[fullKey] && configuration['combine-arrays'])) {
setArg(fullKey, value)
}
}
})
}Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels