sgpinkus
sgpinkus
+1. It should be done. I can't remember exactly why but I don't think its that simple - otherwise it would have been done. Certainly something that would break BWC,...
True. This is a nasty issue. Needs tests though. And want to think about it a bit more before merge.
@MTecknology Will leave open, but won't merge without basic tests.
@bebehei :+1: Absolutely.
@kevinlyles Great list!
> That report will go away the next time I make a change to my website - is there somewhere better to put things like this? Note that while Coveralls.io...
Definitely worth a look. I'm in favor cpan-style too so long as it supports those configuration options. I'm not sure, but it should. Also note I just pushed a branch...
> @sam-at-github Had you been able to work on the testsuite? I'm waiting for you, to merge #74. Merged, but haven't been able to work on test suite. Pretty busy....
@bebehei, @hogarthj Id like to review too please. Will do ASAP today.
Sorry for the noisy commenting - I thought I was doing a review but I was just commenting... LOL. Also need to squash your commits, down to one or two...