npm / cli / #2694

97%

latest: 97%

push

fix(config): consolidate use of npm.color

The config value for `color` should never be directly read by npm, or
its submodules.  The derived value `npm.color` or
`npm.flatOptions.color` is what we want to use.

This PR consolidates the use of these values, makes sure there is only
one place the value is actually calculated, and stops relying on
duplicated code in the setup-log.js file for setting one of them.

PR-URL: https://github.com/npm/cli/pull/3563
Credit: @wraithgar
Close: #3563
Reviewed-by: @lukekarrys

3184 of 3273 branches covered (97.28%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 5 files covered. (100.0%)

5424 of 5567 relevant lines covered (97.43%)

26.61 hits per line