On Mon, Dec 12, 2011 at 1:38 PM, Gerhard Petracek <gpetra...@apache.org> wrote: > +1 for 4 spaces
yup > > regards, > gerhard > > > > 2011/12/12 Shane Bryzak <sbry...@gmail.com> > >> On Mon, Dec 12, 2011 at 9:37 PM, Mark Struberg <strub...@yahoo.de> wrote: >> >> > Hi! >> > >> > I'm a fan of a pretty tight coding convention observation even at build >> > time. >> > >> > What we usually have (in owb and myfaces) is an own 'buildtools' project >> > which contains the checkstyle rules as own artifact. >> > This will then be used in the deltaspike-parent pom as dependency of the >> > maven-checkstyle-plugin. I'll set this up, no worries, easy stuff. >> > >> > The more important thing is to decide _which_ coding conventions we like >> > to follow at all? >> > >> > I have the following suggestions: >> > >> > 1.) no tabs, only spaces! >> > >> >> +1, tabs suck >> >> >> >> > >> > 2.) bracelets on new line? Actually I don't care about >> > > if() >> > > { >> > > dings(); >> > > } >> > or >> > >> > > if() { >> > > dings(); >> > > } >> > but we should only use one stile throughout the whole code. >> > >> > >> >> I don't mind either way here, comfortable with either as long as we pick >> one and are consistent with it. >> >> >> >> > >> > 3.) force bracelets >> > >> > no >> > >> > > if() >> > >> > > dosomething; >> > >> > without bracelets. Instead force: >> > > if() >> > > { >> > >> > > dosomething; >> > > } >> > >> > >> +1 >> >> >> > >> > I'm sure there is a bit more, thus please add the rules which are >> > important for you. >> > (PS: once we found a final solution we should move this into our wiki + >> > provide Eclipse and Idea checkstyle rules. >> > >> >> >> One thing to decide is indent size. Currently in Seam we use 4 spaces, as >> we've recently adopted the JBoss coding standards. Personally, I think >> this is a little too much, previously we had 3 spaces (Gavin's preference) >> which I thought was better. >> >> >> >> > >> > >> > LieGrue, >> > strub >> > >> > >> -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf