On Tuesday 10 November 2015, Kanstantsin Shautsou <[email protected]> wrote:
> JFYI, latest checkstyle version (with latest maven-checkstyle) supports > more flexible configurations like allow inline if’s without brackets. > > Just because you can doesn't mean you should. I favour always using block delimiters for if and else > Different types of inline annotations (configuration 1.3 DTD). > http://checkstyle.sourceforge.net/releasenotes.html > > None of those match "Jesse"-style so hopefully we can kill that abomination off ;-) > > On Nov 10, 2015, at 22:03, Baptiste Mathus <[email protected] > <javascript:_e(%7B%7D,'cvml','[email protected]');>> wrote: > > Well, yeah, but I guess the thing is: > > > "We *try* to *loosely* follow" > > Might have left a lot (too much?) of room for debate :). > Le 10 nov. 2015 6:26 PM, <[email protected] > <javascript:_e(%7B%7D,'cvml','[email protected]');>> a > écrit : > >> > My proposal is to chose some coding style ... >> >> I thought the project already had a style guide; one has already been >> published for the project, anyway. The following appears in the >> "contributing" wiki page, originally authored March 2011 with latest >> revision in August 2015. This page can be reached by following links from >> the Jenkins homepage. >> >> "In the core, we try to loosely follow the Oracle Java Code >> Conventions."[1] >> >> Is this not (or no longer) accurate? If not, this verbiage is misleading >> and should be changed. >> >> [1] https://wiki.jenkins-ci.org/display/JENKINS/contributing >> >> >> >> Christopher Simons >> >> -- >> You received this message because you are subscribed to the Google Groups >> "Jenkins Developers" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected] >> <javascript:_e(%7B%7D,'cvml','jenkinsci-dev%[email protected]');> >> . >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/jenkinsci-dev/d0a010d2-54c6-4b24-a1d5-76aa4692c3fe%40googlegroups.com >> <https://groups.google.com/d/msgid/jenkinsci-dev/d0a010d2-54c6-4b24-a1d5-76aa4692c3fe%40googlegroups.com?utm_medium=email&utm_source=footer> >> . >> For more options, visit https://groups.google.com/d/optout. >> > > -- > You received this message because you are subscribed to a topic in the > Google Groups "Jenkins Developers" group. > To unsubscribe from this topic, visit > https://groups.google.com/d/topic/jenkinsci-dev/8fjvXGYbFJ4/unsubscribe. > To unsubscribe from this group and all its topics, send an email to > [email protected] > <javascript:_e(%7B%7D,'cvml','jenkinsci-dev%[email protected]');> > . > To view this discussion on the web visit > https://groups.google.com/d/msgid/jenkinsci-dev/CANWgJS6jf9JYzrezqRLobmyz13qmyxUGm5XjxPrdD8zcxxC_CQ%40mail.gmail.com > <https://groups.google.com/d/msgid/jenkinsci-dev/CANWgJS6jf9JYzrezqRLobmyz13qmyxUGm5XjxPrdD8zcxxC_CQ%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > For more options, visit https://groups.google.com/d/optout. > > > > -- > You received this message because you are subscribed to the Google Groups > "Jenkins Developers" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected] > <javascript:_e(%7B%7D,'cvml','jenkinsci-dev%[email protected]');> > . > To view this discussion on the web visit > https://groups.google.com/d/msgid/jenkinsci-dev/B95A8E77-94F7-4E9D-8253-08CF6971019B%40gmail.com > <https://groups.google.com/d/msgid/jenkinsci-dev/B95A8E77-94F7-4E9D-8253-08CF6971019B%40gmail.com?utm_medium=email&utm_source=footer> > . > For more options, visit https://groups.google.com/d/optout. > -- Sent from my phone -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/CA%2BnPnMxoJR7Wsw0ySWPVzuHsvCw6N0-6mUDNQMKOp%3D7ydwuP9w%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
