Re: svn commit: r1744731 - /commons/proper/codec/trunk/pom.xml

2016-05-20 Thread sebb
On 20 May 2016 at 18:26, Gary Gregory wrote: > On Fri, May 20, 2016 at 10:22 AM, sebb wrote: > >> On 20 May 2016 at 18:09, Gary Gregory wrote: >> > I've seen some projects that run checkstyle early in the build and causes >> > the build to fail if there are violations. >> > >> > That seems like

Re: svn commit: r1744731 - /commons/proper/codec/trunk/pom.xml

2016-05-20 Thread Gary Gregory
On Fri, May 20, 2016 at 10:22 AM, sebb wrote: > On 20 May 2016 at 18:09, Gary Gregory wrote: > > I've seen some projects that run checkstyle early in the build and causes > > the build to fail if there are violations. > > > > That seems like a nice way to avoid piling up violations over time, an

Re: svn commit: r1744731 - /commons/proper/codec/trunk/pom.xml

2016-05-20 Thread sebb
On 20 May 2016 at 18:09, Gary Gregory wrote: > I've seen some projects that run checkstyle early in the build and causes > the build to fail if there are violations. > > That seems like a nice way to avoid piling up violations over time, and > then spending time fixing them which is boring. > > I

Re: svn commit: r1744731 - /commons/proper/codec/trunk/pom.xml

2016-05-20 Thread Gary Gregory
I've seen some projects that run checkstyle early in the build and causes the build to fail if there are violations. That seems like a nice way to avoid piling up violations over time, and then spending time fixing them which is boring. I think we should do that. Thoughts? Gary On Fri, May 20,