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 <[email protected]> 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" <[email protected]> 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 < > [email protected]> > > 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 > > > > > > > > >
