Perl-Critic / Test-Perl-Critic

Run Perl::Critic as a unit test
http://perlcritic.com
Other
9 stars 7 forks source link

Instantiate Perl::Critic only once #11

Closed holcapek closed 6 years ago

petdance commented 6 years ago

Slick. I like it.

holcapek commented 6 years ago

I squashed couple of commits into one, since one without the others wouldn't make sense. The change is pretty small, anyway.

petdance commented 6 years ago

I'd like to get this merged, but the test fails under Travis. Can you please look into it?

petdance commented 6 years ago

And, are we sure there's no ill effects of reusing the singleton?

holcapek commented 6 years ago

@petdance As for the test failing under Travis: it actually fails also on my local computer, but unfortunately, I didn't take it seriously. Moreover, it fails also with no changes to the upstream, as you can see at PR https://github.com/Perl-Critic/Test-Perl-Critic/pull/13

I don't know what's wrong, I'll keep you posted if I find the cause.

petdance commented 6 years ago

Can I close this and let you create a new PR when you've got it going?

holcapek commented 6 years ago

No problem, let's close it for now.

holcapek commented 6 years ago

Hi @petdance , any chance this could be merged in provided the tests has been fixed previously? Thanks!

pali commented 6 years ago

@petdance: can you look at this change?

petdance commented 6 years ago

Done. It's in the just-pushed-to-CPAN 1.04. Thanks for the nudge.

holcapek commented 6 years ago

@petdance Thanks!