Touching every file of the code would also be a good opportunity to switch from tab to space indention. So if we enforce a strict style, we could also address this issue which causes discussions every now and then.
2015-03-16 21:53 GMT+01:00 Aljoscha Krettek <aljos...@apache.org>: > No, but I don't know whether that's possible. > > The style guide prescribes, for example, this: > > def foo( > a: Int, > b: String, > c: String) > > for methods with long parameter lists while a lot of people do this: > > def foo(a: Int, > b: String, > c: String) > > (IntelliJ also does this). > > The scalastyle rules I added supposedly check for the official scala > guide style but they allow both styles of methods. > > On Mon, Mar 16, 2015 at 5:37 PM, Till Rohrmann <trohrm...@apache.org> > wrote: > > Do we already enforce the official Scala style guide strictly? > > > > On Mon, Mar 16, 2015 at 4:57 PM, Aljoscha Krettek <aljos...@apache.org> > > wrote: > > > >> I'm already always sticking to the official Scala style guide, with the > >> exception of 100 line length. > >> On Mar 16, 2015 3:27 PM, "Till Rohrmann" <trohrm...@apache.org> wrote: > >> > >> > +1 for stricter Java code styles. I haven't looked into the Google > Code > >> > Style but maybe we make it easier for new contributors if we apply a > >> coding > >> > style which is somehow known. > >> > > >> > +1 for line length of 100 for Scala code. I think it makes code > review on > >> > GitHub easier. > >> > > >> > For the Scala style, we could stick to official style guidelines [1]. > >> > > >> > [1] http://docs.scala-lang.org/style/ > >> > > >> > On Mon, Mar 16, 2015 at 3:06 PM, Hermann Gábor <reckone...@gmail.com> > >> > wrote: > >> > > >> > > +1 for the stricter Java code styles. > >> > > > >> > > We should not forget about providing code formatter settings for > >> Eclipse > >> > > and Intellij IDEA (as mentioned above). > >> > > That would help a lot. > >> > > > >> > > (Of course if we'll use Google Code Style, they already provide such > >> > files > >> > > < > >> > > > >> > > >> > https://code.google.com/p/google-styleguide/source/browse/trunk/intellij-java-google-style.xml > >> > > > > >> > > .) > >> > > > >> > > On Mon, Mar 16, 2015 at 2:45 PM Alexander Alexandrov < > >> > > alexander.s.alexand...@gmail.com> wrote: > >> > > > >> > > > +1 for not limiting the line length. > >> > > > > >> > > > 2015-03-16 14:39 GMT+01:00 Stephan Ewen <se...@apache.org>: > >> > > > > >> > > > > +1 for not limiting the line length. Everyone should have a good > >> > sense > >> > > to > >> > > > > break lines. When in exceptional cases people violate this, it > is > >> > > usually > >> > > > > for a good reason. > >> > > > > > >> > > > > On Mon, Mar 16, 2015 at 2:18 PM, Maximilian Michels < > >> m...@apache.org> > >> > > > > wrote: > >> > > > > > >> > > > > > +1 for enforcing a more strict Java code style. However, let's > >> not > >> > > > > > introduce a line legth of 100 like in Scala. I think that's > >> hurting > >> > > > > > readability of the code. > >> > > > > > > >> > > > > > On Sat, Mar 14, 2015 at 4:41 PM, Ufuk Celebi <u...@apache.org> > >> > wrote: > >> > > > > > > >> > > > > > > On Saturday, March 14, 2015, Aljoscha Krettek < > >> > aljos...@apache.org > >> > > > > >> > > > > > wrote: > >> > > > > > > > >> > > > > > > > I'm in favor of strict coding styles. And I like the > google > >> > > style. > >> > > > > > > > >> > > > > > > > >> > > > > > > +1 I would like that. We essentially all agree that we want > >> more > >> > > > > > > homogeneity and I think strict rules are the only way to go. > >> > Since > >> > > > this > >> > > > > > is > >> > > > > > > a very subjective matter it makes sense to go with something > >> > > > (somewhat) > >> > > > > > > well > >> > > > > > > established like the Google code style. > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > >> > > >> >