> I think it is good to have control over what is actually checked by > checkstyle. Having a customised version is quite common practice also in > other commons components. >
I would have preferred you would have discussed first before changing configurations. Control and customizations, if needed, can be included in the checkstyle suppressions file. Please revert pom configuration, if applied, and discuss first in a separated thread. > The one I checked in is from commons math, where I disabled some checks > for now (e.g. @version, license header). The math style is not the one we've been using, if the configuration has to be imported, then it has to be from Maven, wich is the one we've been using. http://people.apache.org/~simonetripodi/ http://simonetripodi.livejournal.com/ http://twitter.com/simonetripodi http://www.99soft.org/ On Sat, Mar 3, 2012 at 2:42 PM, Thomas Neidhart <[email protected]> wrote: > On 03/03/2012 02:36 PM, Simone Tripodi wrote: >> Thomas, >> >> checkstyle is not needed because we are using the one brought by the >> plugin (the maven one), you can safety drop it. > > I think it is good to have control over what is actually checked by > checkstyle. Having a customised version is quite common practice also in > other commons components. > > The one I checked in is from commons math, where I disabled some checks > for now (e.g. @version, license header). > > Thomas > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
