search

quattor / maven-tools

Maven-based Build Tools
www.quattor.org
Apache License 2.0
3 stars 12 forks source link

Test::Quattor::Critic: add Perl::Critic whitelist testing #115

Closed stdweird closed 7 years ago

stdweird commented 7 years ago

Also adds an informational perltidy test Requires #114

stdweird commented 7 years ago

@jrha and you thought #112 would cause lots of work 😁

stdweird commented 7 years ago

also, please don't make any comments on what critic policies we should add and what not. if you want certain tests, open a PR, set a milestone and check your agenda when you have time to fix the fallout

jrha commented 7 years ago

:see_no_evil:

stdweird commented 7 years ago

The perltidy test is way too difficult to tune, and perltidy has no support for whitelisting. You are advised to look at the output and we should discuss final parameters and possible impact in followup PRs