If Sonar (whatever that is) already runs it, then it may be nice to add this
to the coding-guidelines page; http://cxf.apache.org/coding-guidelines.html.
... lets the user community know the software quality level you are at. :)
-- Robert
--------------------------------------------------
From: "Benson Margulies" <[email protected]>
Sent: Thursday, October 14, 2010 8:34 PM
To: <[email protected]>
Subject: Re: Apache CXF and FindBugs
Let's slow down a bit on this idea and discuss it on the dev list.
FindBugs has a very, very, very broad definition of a bug. I bet there
are cases where checkstyle and findbugs *disagree* about some source
construct.
Things it complains of should be talked over on dev before anyone
bothers to make patches.
Our Sonar instances already runs it, I believe, so you can see what it
complains about already.
On Thu, Oct 14, 2010 at 5:27 PM, Daniel Kulp <[email protected]> wrote:
On Thursday 14 October 2010 10:53:17 am Robert Liguori wrote:
I see that CXF makes use of PMD and Checkstyle (
http://cxf.apache.org/coding-guidelines.html).
Any reason why CXF doesn't make use of FindBugs? -
http://findbugs.sourceforge.net/
Most likely, no one really pursued it. Again, fixing issues that it
finds is
another valuable contribution.
--
Daniel Kulp
[email protected]
http://dankulp.com/blog