Beefy Boxes and Bandwidth Generously Provided by pair Networks
The stupid question is the question not asked

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 all is quiet...

How do I use this? | Other CB clients
Other Users?
Others imbibing at the Monastery: (4)
As of 2018-02-18 09:46 GMT
Find Nodes?
    Voting Booth?
    When it is dark outside I am happiest to see ...

    Results (253 votes). Check out past polls.