The thing is.. we will have more failures on the PR builds.. .I would rather have people getting issues at their code.
Perhaps you could just document how to disable on README.. and make it easy for users to disable it. On Wed, May 6, 2015 at 10:14 AM, Andy Taylor <andy.tayl...@gmail.com> wrote: > Personally I prefer it just to be run on PR's, not everyone that build from > source is bothered about it. I think users should be able to build from > source no matter what they have in their source tree, log files, users own > source etc. > > > On 06/05/15 14:54, Clebert Suconic wrote: >> >> So, the poms now have the RAT check enabled. That means we would get >> our own mistakes instead of wasting server's time resource with a >> failed PR. >> I think that's the right way to do it... we wouldn't waste server's >> time resource on failed PRs... and wouldn't waste anyone's time on >> looking at PRs for that kind of error. >> >> So, I'm strongly in favor on keeping the check on on the builds. >> >> >> Is there anyone with a different opinion on this.. and on that case,, why? >> >> >> >> On Tue, May 5, 2015 at 7:57 PM, Clebert Suconic >> <clebert.suco...@gmail.com> wrote: >>> >>> Just a heads up, >>> >>> >>> Justin Bertram has enabled license checks on the build again, and the >>> PR checks... >>> >>> >>> So, from now on committing java files without headers won't get un >>> noticed without a build failure ;) >> >> >> >> > -- Clebert Suconic http://community.jboss.org/people/clebert.suco...@jboss.com http://clebertsuconic.blogspot.com
