While it doesnt change all the codebase and it makes the build - mvn -failing if not respected +1. Le 19 nov. 2015 07:31, "Alex Soto" <asot...@gmail.com> a écrit :
> I think that a good way to maintain all code style using the same format > would be use a tool like Walkmod (http://walkmod.com/) it is only an ida > of > course. > > Alex > > El dj., 19 nov. 2015 a les 16:29, Romain Manni-Bucau (< > rmannibu...@gmail.com>) > va escriure: > > > Go for it > > Le 19 nov. 2015 06:54, "Thiago Veronezi" <thi...@veronezi.org> a écrit : > > > > > Hi guys, > > > > > > is anyone going to commit the files with the wrong line endings? > > > > > > []s, > > > Thiago. > > > > > > > > > On Thu, Nov 19, 2015 at 9:38 AM, Romain Manni-Bucau < > > rmannibu...@gmail.com > > > > > > > wrote: > > > > > > > +1 > > > > > > > > Only note: we have a huge codebase so we shouldnt completely reformat > > it > > > > IMO. Said otherwise we should ensure the IDE reformatting keeps code > as > > > it > > > > is in 99% of the case > > > > > > > > Ps: thks a lot for the github link! > > > > Le 19 nov. 2015 05:19, "Thiago Veronezi" <thi...@veronezi.org> a > > écrit : > > > > > > > > > Formating >> > > > > > > > > > > When I introduced the pmd and checkstyle plugins, I've created a > new > > > > maven > > > > > profile in order to keep the simple "mvn clean install" a bit > faster. > > > > Maybe > > > > > we could make it run by default without the extra profile? This > way, > > a > > > > > developer would know right away when she/he introduced any > formatting > > > > > violation without depending on any IDE. > > > > > > > > > > []s, > > > > > Thiago. > > > > > > > > > > > > > > > On Thu, Nov 19, 2015 at 7:43 AM, Andy Gumbrecht < > > > > agumbre...@tomitribe.com> > > > > > wrote: > > > > > > > > > > > Hi everyone, > > > > > > > > > > > > We keep running into PMD and formatting issues, as well as line > > > ending > > > > > > issues. These are just some ideas I'd like to run by you all, but > > > > please > > > > > > check the line-ending doc. Whatever we decide upon needs to be > > > applied > > > > > asap. > > > > > > > > > > > > Line Endings > > > > > > ----------------- > > > > > > Line ending issues need to be resolved by everyone using this > > > > > > documentation (for your OS of choice): > > > > > > https://help.github.com/articles/dealing-with-line-endings/ - as > > > > > probably > > > > > > 99.9% of us are GitHub users, this should be the rule. > > > > > > > > > > > > Please feel free to add to and extend the attributes with > anything > > > that > > > > > > may be missing, but do not remove the basic structure of: > > > > > > https://github.com/apache/tomee/blob/master/.gitattributes > > > > > > > > > > > > Formatting > > > > > > --------------- > > > > > > Basically, pressing CTRL+ALT+L (Reformat) should result it the > same > > > for > > > > > > everyone that is hacking on TomEE. This includes all project > files. > > > > > > > > > > > > The only way this is going to work is if we define a rigid guide > > or, > > > > even > > > > > > better, provide an Intellij-IDEA settings import/export file for > > > > everyone > > > > > > to use as a foundation. > > > > > > > > > > > > David, as the project lead and our glorious master of all things > > > magic, > > > > > it > > > > > > would be great if you could provide this. I'm going to update the > > > site > > > > a > > > > > > bit, so would be great to get this added as soon as possible. > > > > > > > > > > > > Once we have this then a global reformat should be possible > across > > > the > > > > > > entire project base. Enabling and using @formatter:off/on should > be > > > > used > > > > > to > > > > > > protect critical sections. > > > > > > > > > > > > Feedback welcome. > > > > > > > > > > > > Andy. > > > > > > > > > > > > -- > > > > > > Andy Gumbrecht > > > > > > https://twitter.com/AndyGeeDe > > > > > > http://www.tomitribe.com > > > > > > > > > > > > > > > > > > > > > > > > > > >