search

SamVerschueren / clinton

Project style linter
MIT License
122 stars 9 forks source link

Merge cli options with config file options? #14

Closed SamVerschueren closed 7 years ago

SamVerschueren commented 8 years ago

Currently, the rules provided via the cli and the rules provided via a config file are merged where the cli options have higher priority. Should we do that or should we implement that if cli options are provided, the config file options are dropped?

sindresorhus commented 8 years ago

Merge. That's how I do it in AVA and XO. The benefit is that it let's you define a base config in package.json and then override with CLI flags. You should recommend people use the config whenever possible for maintainability and to make it easier for eventual other tools to read the config.