Don't overrider NODE_V8_COVERAGE

[kilianc]

Is it possible to prepopulate --temp-directory with the content of $NODE_V8_COVERAGE if set? I am trying to integrate c8 in my workflow and I really like the ability to separate execution and reporting.

• NODE_V8_COVERAGE=/tmp/cov npm test
• NODE_V8_COVERAGE=/tmp/cov c8 report

Happy to contribute id accepted.
Cheers!

[shinnn]

Good idea. PR welcome.

The patch would mainly update the following two lines:

• c8/lib/parse-args.js

  Line 64 in 5ea4313

  default: './coverage/tmp',

  Use process.env.NODE_V8_COVERAGE as a fallback if it exists, instead of './coverage/tmp'

• c8/bin/c8.js

  Line 29 in 5ea4313

  process.env.NODE_V8_COVERAGE = argv.tempDirectory

  Don't override NODE_V8_COVERAGE if it exists

[bcoe]

@kilianc can you confirm that this is now working for you?