gitlabhq/gitlabhq — pull request #7107 by cirosantilli (2014-06-10 14:37:05+00:00)
Title: Clarify that bbastov is the style of Hound CI 2.

Added Rubocop link.

Set option B, previous line.

---

Since I wrote this, Hound docs improved. [They say their default is](https://github.com/thoughtbot/guides/tree/master/style#formatting):

> If you break up a chain of method invocations, keep each method invocation on its own line. Place the . at the end of each line, except the last.

I don't think bbastov specifies one method per line, so in theory Hound should check more like "Option B + one method per line."

In practice however hound [does not seem to enforce one method by line by default](https://github.com/cirosantilli/test/pull/9#discussion-diff-13595616R36), so it ends up being the same as bbastov because of that bug.

For now we can leave Option B.

For the future I propose we use one single tool and the style that comes with it to reduce the probability of conflicts between the styles, so either:

- Hound + [hound style](https://github.com/thoughtbot/guides/tree/master/style#ruby)
- Rubocop + [bbastov style](https://github.com/bbatsov/ruby-style-guide)