Hi folks,
short time after asking [1], people at infrastructure made a custom ruleset for Cocoon at Sonar [2] and we have no more Blocker or Critical violations.

Kudos to the infrastructure team!

[1] https://issues.apache.org/jira/browse/INFRA-3828
[2] https://analysis.apache.org/rules_configuration/index/10

On 05/08/2011 11:57, Steven Dolg wrote:
Am 05.08.2011 11:25, schrieb Nathaniel, Alfred:
Hi Francesco,

I am a big fan of Findbugs and persueing a zero-warnings policy.
In other projects we use a Findbugs exclude filter file to
disable warnings and document the deliberation for doing so.

That's what we do as well.
TBH, the default ruleset contains more than one rule that I find highly suspicious if not outright stupid.
Zero warnings is IMO the only way to go, same as with unit tests.


Sonar should foresee to pull such an exclude filter from the
project's code base.

NB we also exclude XFB_XML_FACTORY_BYPASS with reason
"axis.client.Stub requires specific axis.message.SOAPHeaderElement".

Maybe we should try to get our own set of rules (there are rulesets "CXF Rules with Findbugs" and "My Faces with Findbugs", so I guess that's possible).

I think it's easier to start with a more lenient ruleset, work (close) to 0 warnings and then add a couple more restrictions and repeat. Sitting in front of hundreds of warnings can be a frustrating situation...

Steven


Cheers, Alfred.

-----Original Message-----
From: Francesco Chicchiriccò [mailto:ilgro...@apache.org]
Sent: Freitag, 5. August 2011 08:50
To: dev@cocoon.apache.org
Subject: Filtering out (some) Sonar violations

...

My proposal would be to open a issue to INFRA in order to ask for
disabling these two rules: do you see any problem, with this?

For my personal experience about Findbugs, PMD and Checkstyle, it is
rather impossible to satisfy all the rules (thus bringing the number of
violations to zero); however, I personally think that reducing the huge
amount that we have at the moment would dramatically improve Cocoon 3's
source code quality.

...

The content of this e-mail is intended only for the confidential use of the person addressed. If you are not the intended recipient, please notify the sender and delete this e-mail immediately.
Thank you.
--
Francesco Chicchiriccò

Apache Cocoon Committer and PMC Member
http://people.apache.org/~ilgrosso/

Reply via email to