[hibernate-dev] Checkstyle: enforced!

Hardy Ferentschik hardy at hibernate.org
Thu Apr 18 12:22:01 EDT 2013


On 18 Jan 2013, at 6:15 PM, Sanne Grinovero <sanne at hibernate.org> wrote:

> On 18 April 2013 17:12, Davide D'Alto <daltodavide at gmail.com> wrote:
>> I don't have any particular objection to set it on the verify phase. I tend
>> to always use mvn install anyway.
>> 
>> The argument I can use to promote the compile phase (for sake of discussion
>> only)
>> I like the fact that I can check the code writing mvn compile, therefore
>> without starting any test.
>> If you think you are going to prototype something big you can always you
>> can always change the attribute failsOnError to false.
> 
> right but if this motivates people to hardcode -DfailsOnError-false in
> their scripts, we're back to square one ;-)

bottom line is that one of the core devs will integrate the pull request and as part
of it verify/test it. If the contributor did fail to match the style we can respond and 
hopefully we even automate the pull request build. 

--Hardy




More information about the hibernate-dev mailing list