BlockAlignment on chaining methods

[lenntt]

I have similar code that results in a BlockAlignment failure:

my_object.chaining_this_very_long_method(with_a_parameter)
    .and_one_with_a_block do
  do_something
end

Note, this passes:

my_object.chaining_this_very_long_method(
    with_a_parameter).and_one_with_a_block do
  do_something
end

While I prefer to write down the first.

[jonas054]

The reason why this is reported as an offence is the implementation from #362, which was a fix for #346. I'm starting to think that it was a mistake to introduce that change. The Style Guide says that "multiline chaining is always ugly" and multiline chaining is exactly what #346 was about.

I propose that we remove the handling introduced in #362. That will solve this issue.

[edzhelyov]

I also think that multiline chaining should be avoided.

[lenntt]

Some Ruby DSLs, like RSpec, are pretty much about chaining. Therefore, I'll probably end up keeping this cop disabled, which is fine by me.

[fuadsaud]

@SirLenz0rlot had to do the same. But I disagree with the guide. Multi-line chaining, IMO, looks totally ok if you're using {} blocks instead of do/end. But the guide also discourages multi-line {} blocks, which is another silly decision in my opinion - I understand why these decisions were made, but I heavily disagree with them. Disabling theses cops is the way to go.