myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthias Wessendorf" <mwessend...@gmail.com>
Subject Re: PMD on MyFaces
Date Thu, 23 Feb 2006 21:01:03 GMT
I think BSD makes no problem



On 2/23/06, Sean Schofield <sean.schofield@gmail.com> wrote:
> Interesting idea.  I think for the next few days we will have our
> hands full with the commons refactor but I'm open to trying it out.
> Not sure if the license matters since we are not actually distributing
> the plugin.  Maven is the one downloading it.
>
> Sean
>
> On 2/23/06, Jurgen Lust <Jurgen.Lust@ugent.be> wrote:
> > Now that we're using maven2 to build MyFaces, perhaps it's time to take
> > advantage of some of the more interesting plugins, and specifically the
> > PMD plugin:
> > http://pmd.sourceforge.net/
> >
> > The PMD plugin for maven2 offers 2 report types, a Cut&Paste Detector
> > and a problem detector, with configurable rulesets. I've introduced this
> > plugin at work about a month ago and I must say that it has definitely
> > improved the code quality of our projects. Even very experienced Java
> > developers can learn something from this auditing tool.
> >
> > Do you think it would be interesting to put this plugin in the MyFaces
> > POM? We could start with a limited ruleset to not get overwhelmed in the
> > beginning, and then gradually expand the list of rulesets. I'm willing
> > to lead this auditing effort, and implement the changes suggested by PMD.
> >
> > Of course, I could just do this audit without the PMD maven plugin, but
> > using the Eclipse PMD plugin as well, but then noone would see the
> > progress we're making in that area :) One reason not to use this plugin
> > might be the license of PMD (BSD license), does the license of the maven
> > plugins matter for an Apache project?
> >
> > Kind regards,
> >
> > Jurgen
> >
>


--
Matthias Wessendorf
Zülpicher Wall 12, 239
50674 Köln
http://www.wessendorf.net
mwessendorf-at-gmail-dot-com

Mime
View raw message