Long chains of method calls should be formatted one per line

Input

Person
  .select('people.id, people.name, comments.text')
  .joins(:comments)
  .where('comments.created_at > ?', 1.week.ago)

Current output

Person.select('people.id, people.name, comments.text').joins(:comments).where(
  'comments.created_at > ?', 1.week.ago
)

Expected output

Person
  .select('people.id, people.name, comments.text')
  .joins(:comments)
  .where('comments.created_at > ?', 1.week.ago)

or maybe

Person.select('people.id, people.name, comments.text')
      .joins(:comments)
      .where('comments.created_at > ?', 1.week.ago)

I’d be willing to help implement this change if we decide it’s a good idea, however I’d need a little guidance on how to go about doing that.

[kddnewton]

Hot damn. This is finally closed. It’ll be out in the next release.

I’d imagine there will be more follow up on it, but we can open new issues.

[AluisioASG]

It seems like this could help with another case:

Input

requested_locale_tags.map { |locale_tag| TwitterCldr::Shared::Locale.parse(locale_tag) }.reject { |locale| locale.language == 'und' }

Current Output

requested_locale_tags.map do |locale_tag|
  TwitterCldr::Shared::Locale.parse(locale_tag)
end.reject { |locale| locale.language == 'und' }

Expected output

requested_locale_tags
  .map { |locale_tag| TwitterCldr::Shared::Locale.parse(locale_tag) }
  .reject { |locale| locale.language == 'und' }

Using the expected output as input yields the same result.