On 23 March 2016 at 19:07, Davide D'Alto <daltodav...@gmail.com> wrote: > Yeah, go with it. > I don't think it is going to be too hard to fix the conflicts in the Neo4j > module.
Ok, thanks: - https://hibernate.atlassian.net/browse/OGM-1001 > >>TLRD: > > Did you mean: TLDR; probably. Or make you doubt, so to make you read it all.. Sanne > > > On Wed, Mar 23, 2016 at 7:02 PM, Gunnar Morling <gun...@hibernate.org> > wrote: >> >> +1 for fixing these things, I also was wondering about some apparent style >> violations which CS failed to report. Must have been these bug fixes then. >> >> The only thing coming to my mind is Davide's work which may change things >> in the existing Neo4j code. Maybe you two quickly sync? Apart from that >> I'd >> say let's do it. >> >> 2016-03-23 19:57 GMT+01:00 Sanne Grinovero <sa...@hibernate.org>: >> >> > TLRD: don't be surprised if I'll be sending some rather large code >> > style fixes. Not wasting my time nor intending to waste yours.. >> > >> > >> > I've been writing quite some code enjoying Java 8 specific features in >> > the new OGM module, >> > as I assumed this would not be an issue at all since Hot Rod requires >> > Java8 and we require Java8 to build anyway. >> > >> > Unfortunately the Checkstyle plugin crashes when it encounters Java8 >> > bytecode ops, something which is easily fixed with a plugin upgrade. >> > The plugin upgrade however *fixed* some bugs in Checkstyle, for which >> > it had been accepting some syntax even though we were explicitly >> > intending to guard against these. >> > >> > So now there are several (~300) style failures; these are legitimate >> > failures so my plan is to fix these. >> > >> > If someone is busy with some rather large refactoring, please speak up >> > now as I can wait with this, or we can stagger this cleanup by sending >> > the fixes for some modules only, and the remaining later. >> > >> > P.S. I'd rather cleanup the bad code than revert my Java8 code.. both >> > as a lesser effort, and as a better ultimate result. >> > >> > Thanks, >> > Sanne >> > _______________________________________________ >> > hibernate-dev mailing list >> > hibernate-dev@lists.jboss.org >> > https://lists.jboss.org/mailman/listinfo/hibernate-dev >> > >> _______________________________________________ >> hibernate-dev mailing list >> hibernate-dev@lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/hibernate-dev > > _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev