coverage does too much file reading #27

Closed
isaacs opened this Issue Dec 6, 2011 · 1 comment

Projects

None yet

1 participant

@isaacs
Member
isaacs commented Dec 6, 2011

The coverage thing does this right now:

  1. Read all files in the --cover dir
  2. attach the coverage hook to apply to /.*/

This is unnecessarily costly. Also, it's using a bunch of sync io, which needs to be factored out.

It'd be better to just apply the coverage hook to the --cover dir, so it works without the second read.

@isaacs
Member
isaacs commented May 12, 2015

Not using that coverage mechanism any more. nyc will be providing coverage soon.

@isaacs isaacs closed this May 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment