Alex Seville

Results 59 comments of Alex Seville

I think this solution is good. The `use strict` statement doesn't need to be explicitly covered, although I'm curious if it will have an effect on the line reporting...

I noticed the tests, I meant moreso compatibility with the existing default reporter. I think the coverage stats may unfortunately be somewhat strongly binded to the source line counts On...

It may not be a problem at all, it's just the only area of concern I'd have. Otherwise it's a great fix, thanks! On Mon, Oct 14, 2013 at 11:04...

Nice, thanks. On Mon, Oct 14, 2013 at 11:06 PM, jaredwy [email protected] wrote: > I will invest some time to put your fears to bed, then open the pull >...

I don't think he ever made a PR, but also, might not be the best idea to use an unmaintained (for a few years now) software package for your company's...

This seems like a good feature. I'm running Travis on the PR again but I think there are merge conflicts now too. Sorry for the delay.

Not sure why it failed before, but now there are merge conflicts on the PR

If the merge conflict is fixed I think this would be great to merge. Sorry for the delay.

I've published a new version, 1.2.2. Let me know if it changes anything.

Hmm, it's working on travis CI at [email protected] - https://travis-ci.org/alex-seville/blanket/jobs/110242642