Beefy Boxes and Bandwidth Generously Provided by pair Networks
Clear questions and runnable code
get the best and fastest answer

Re^3: Module Announcement: Perl-Critic-1.01

by xdg (Monsignor)
on Jan 26, 2007 at 15:13 UTC ( #596722=note: print w/replies, xml ) Need Help??

in reply to Re^2: Module Announcement: Perl-Critic-1.01
in thread Module Announcement: Perl-Critic-1.01

However, the web-service is currently configured for maximum strictness. I suppose I should loosen that up a bit.

I suspect you'll get more positive reactions if you tune the level of strictness for what you think "typical" (hah!) programmers would find helpful. Not everyone takes PBP as gospel.

Alternatively, even if you don't expose all the configuration via the service, perhaps either having two options "Analyze my code aggressively" or "Analyze my code mildly" would help. Or perhaps just grouping the output in descending order of severity -- so people see the highest severity issues first and then can decide where to stop paying attention.


Code written by xdg and posted on PerlMonks is public domain. It is provided as is with no warranties, express or implied, of any kind. Posted code may not have been tested. Use of posted code is at your own risk.

  • Comment on Re^3: Module Announcement: Perl-Critic-1.01

Replies are listed 'Best First'.
Re^4: Module Announcement: Perl-Critic-1.01
by educated_foo (Vicar) on Jan 28, 2007 at 18:30 UTC
    Agreed! I tried out the web version on one of my modules, and instantly dismissed it as (1) needless pedantry, (2) uninformative, and (3) pimping PBP:
    Code is not tidy at line 1, column 1. See page 33 of PBP. Severity: +1
    Subsequent lines did not improve my opinion of the module.

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://596722]
and monks are getting baked in the sun...

How do I use this? | Other CB clients
Other Users?
Others musing on the Monastery: (5)
As of 2017-04-26 16:33 GMT
Find Nodes?
    Voting Booth?
    I'm a fool:

    Results (484 votes). Check out past polls.