On Mon, Oct 26, 2015 at 12:44 PM, Kanstantsin Shautsou <[email protected]> wrote: >> I have never had any problem understanding anyone else’s code because of >> their formatting choices > > Because you know very well this code base
How do you think I got to know (some of) the code base in the first place? By fixing problems, and digging through history to see when and why those problems were introduced. > See example of automated check > https://github.com/jenkinsci/envinject-plugin/pull/73#discussion_r40529580 Interesting, though sounds very painful to contribute to since you must first file the PR, then go back and wait for the bot to ask you to reformat. Much better to be able to repeatedly run a style checker tool locally before committing. Not sure if there is any such tool which is able to automatically skip lines already present in the `master` branch or something like that. -- 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/CANfRfr2W0x5Q2NTAaZMQz4G5nvP9-6XTLKbTxP9VkFSvQML%2BvA%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
