The documentation on the http://guides.rubyonrails.org/contributing_to_ruby_on_rails.html was outdated about indenting decision. Let's update that.
Update coding convention from master
Is there any discussion about indenting after private/protected? At the same time I want to stay in convention with Rails', I'd like to see the arguments favoring indent. I honestly prefer to not indent methods after private/protected.
@lunks there is really no discussion going on. Of course this only applies to the Rails code base, Rails applications can have other conventions.
Conventions are arbitrary up to some point. There are people who prefer not to use parens for method definitions, there are people who prefer to surround arguments in a method call with spaces, there are people who put private/protected against the left margin so that they stand out... Though we all have our preferences, at the end of the day it does not matter much, those are the conventions agreed in the project and we stick to them to have a uniform code base.
I understand, but what motivated the change into the private/protected indenting? Please do not think of this as rant, I'm rather curious about what made you guys change your mind about this.
I prefer the older convention, and as far as discussions went on my side so far, there were no compelling arguments to either way. As such, changing this probably was led by someone stating "hey, this is better because of x." and the change was made by @sikachu afterwards.
That was the convention of the project before (https://rails.lighthouseapp.com/projects/8994/source-style) and got somehow changed. Don't remember the details. I believe David himself feels strong about indenting after private/protected in particular.
@lunks it was not a decision reached by consensus among rails core. some of us do not like the intented style, but ultimately it's not something that there's much point arguing about as we have much more important things to do :)
THIS IS NOT WHAT I WANT! I WANT MY CODING STYLE BACK!
Indeed, @jonleighton, not much to talk about other than pick one and live with it.
Thanks for the input. :)