fdesjardins / coverage-viewer

A cross-platform code coverage report generator
MIT License
16 stars 4 forks source link

Add option to serve the coverage report #1

Closed fdesjardins closed 6 years ago

fdesjardins commented 6 years ago

It's kind of a hassle to have to generate the html over and over. You can use nodemon or similar to regenerate the reports when the coverage output changes, then run browsersync to get livereloading...

I think it'd be better to just serve the reports when the user passes a flag to save them the hassle.

coderpatros commented 6 years ago

Pull request #4 means you can use file:// urls and don't need a server.

fdesjardins commented 6 years ago

Yeah, that's a good change, and definitely helps with this problem. I'm not sure I want to drop the idea of having an option to serve the results, if it could be implemented without pulling in extra libs.

In any case, it's now a lower priority if we can get your changes in. :slightly_smiling_face:

fdesjardins commented 6 years ago

Closed in https://github.com/fdesjardins/coverage-viewer/pull/8.