I guess my reason for the patch was that there is inconsistency across
rails, with a mix of then, colon, semicolon, and newline.. any
thoughts on what it should be?

Very much depends on what makes sense in the context. I often use
different conditional styles depending on the particulars. Not using
the same style everywhere is not necessary a bug.
--
David Heinemeier Hansson
http://www.loudthinking.com -- Broadcasting Brain
http://www.basecamphq.com   -- Online project management
http://www.backpackit.com   -- Personal information manager
http://www.rubyonrails.com  -- Web-application framework
_______________________________________________
Rails-core mailing list
Rails-core@lists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-core

Reply via email to