Enable Layout/EmptyLineAfterGuardClause cop

[sambostock]

This enables the Layout/EmptyLineAfterGuardClause cop, which is autocorrectable.

From the documentation

# bad
def foo
  return if need_return?
  bar
end

# good
def foo
  return if need_return?

  bar
end

# good
def foo
  return if something?
  return if something_different?

  bar
end

# also good
def foo
  if something?
    do_something
    return if need_return?
  end
end

[rafaelfranca]

How many violations for this cop do we have in core?

[sambostock]

@rafaelfranca, 8,271 offenses detected, all auto-correctable though

[volmer]

Nitpick but for next time we can simply remove this entry from the config as this cop is enabled by default. We only need to explicitly enable true for recent cops that are still in a "pending" state.

[sambostock]

Good point. I should have removed it and run the tests, which would have revealed the change to the full config.

[miry]

I wonder how it would looks like for case. Example I have bar?, that is part of bar and now it looks like bar? is part of section quux.

def foo
  quux
  quux
  quux
  return if bar?

  bar
end