Re: [hibernate-dev] Checkstyle and ORM

+1 I thought it was in already :) > On 18 mai 2015, at 17:20, Steve Ebersole <steve(a)hibernate.org&gt; wrote: > > Sorry hit send too soon :( > > Anyway.. I assume it is because we know this information from commit > history? > > As far as the ORM one, I propose a similar change. Any objections? > > On Mon, May 18, 2015 at 10:14 AM, Steve Ebersole <steve(a)hibernate.org&gt; > wrote: > >> As far as the file header, I have often thought of condensing our ORM one >> so that is awesome information. For my own curiosity did Richard give you >> a reason that the copyright statement is not necessary? >> >> On Mon, May 18, 2015 at 9:58 AM, Sanne Grinovero <sanne(a)hibernate.org&gt; >> wrote: >> >>>> On 18 May 2015 at 15:23, Emmanuel Bernard <emmanuel(a)hibernate.org&gt; wrote: >>>> Note that custom checkstyle rule implementations does not fare well >>> with IDE checkstyle rules enforcement. >>>> I can't make it reliably work on search and OGM. >>> >>> As far as I remember the problem was related with having the >>> customisations as part of the project which is being built. >>> I'd hope one day we could publish a common set of rules by releasing >>> the custom as an independent project, consumed by all. We could try >>> that already for some projects, and see if the others catch up >>> eventually? They don't even have to use the same XML so there is some >>> room for customizing which rules exactly each project wants to >>> enforce. >>> >>> I'm not sure if that will resolve your problem but it's worth trying; >>> it could also slightly speed up the builds by a couple of seconds, as >>> you wouldn't have to rebuild the checkstyle module each time. >>> _______________________________________________ >>> hibernate-dev mailing list >>> hibernate-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/hibernate-dev > _______________________________________________ > hibernate-dev mailing list > hibernate-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/hibernate-dev