Fixing all the checkstyle errors first is kind of a prerequisite to enabling it by default.
On 31 July 2016 at 15:10, Charles Honton <[email protected]> wrote: > Why wouldn’t we want build to fail early if incorrect style is used? > > chas > > > On Jul 31, 2016, at 11:09 AM, Oliver Heger <[email protected]> > wrote: > > > > Hi, > > > > in revision 1742698 the checkstyle configuration has been changed. The > > log says "fixed checkstyle violations, updated to latest version of > > checkstyle-maven-plugin, ensure correct checkstyle configuration is > > applied in all cases". > > > > I think with the new configuration checkstyle is now run on every mvn > > install and causes the build to fail if there are checkstyle errors. > > This is probably not what we want. What was the reason for this update, > > i.e. under which circumstances was an incorrect checkstyle configuration > > used? > > > > Thanks > > Oliver > > > > --------------------------------------------------------------------- > > 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] > > -- Matt Sicker <[email protected]>
