Skip to content

Clarify that bbastov is the style of Hound CI 2.

gitlab-qa-bot requested to merge github/fork/cirosantilli/clarify-hound-ci-style into master

Created by: cirosantilli

Continued from https://github.com/gitlabhq/gitlabhq/pull/6786

Added Rubocop link.

Set option B, previous line.

Since I wrote this, Hound docs improved. They say their default is:

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, 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:

Merge request reports