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


Reply via email to