Disable rescue alignment cop until bugs are fixed

[zachsabin]

Currently the rescue alignment cop has an issue on variable assignments where it enforces this style:

return_val = begin
  # computation
             rescue
               # rescued value
end

which is bad.

rubocop/rubocop#7531 will maybe fix it, but I think it makes sense to disable this for now until it's fixed in a rubocop release.

[robotpistol]

The solution here is to do this instead

return_val = 
  begin
    # computation
  rescue
    # rescued value
  end

[robotpistol]

The problem is more being caused by inconsistent lining up of en with begin block and the body. This isn't a bug but rather configuration

[zachsabin]

True, but we'd like the option to have rescue align with the variable assignment. It seems like that configuration is in the works, in the meantime the auto correct (what I posted in the PR body) is a very strange format that we'd like to avoid.

[Umofomia]

According to this, it looks like there is a bug with the associated autocorrect too: rubocop/rubocop#6254

I'm fine with disabling this until the cop is more stable.

[zachsabin]

I think the autocorrect bug is now fixed.

I'm going to hold off on merging this for a little bit to see if the rubocop configuration change gets merged. If it seems like it will take a while, then I'll merge this to disable the cop.

[BMorearty]

Ship it

[michael-zoller-airbnb]

the bug has been fixed as of rubocop version 0.9.1 which adds the BeginEndAlignment cop with the default "start_of_line"

[pariser]

Since we've progressed well past rubocop 0.9.1, I'm going to close this out. Thanks everyone for the contributions.