Rubocop: An option for Lint/HandleExceptions to accept comments

[dduugg]

Is your feature request related to a problem? Please describe.

I find that Lint/HandleExceptions is often bypassed with a nil in the rescue block. This is not helpful, whereas comments would be. However, a rescue block consisting of only a comment will still trigger a Lint/HandleExceptions offense.

Describe the solution you'd like

I would like an option to allow comments within a rescue block to avoid Lint/HandleExceptions. Perhaps AllowComments: true.

Describe alternatives you've considered

• Instead of a config option, allow comments in all cases
• Disable the cop
• Also ban rescue blocks consisting of nil with no add'l context
• Continue with status quo

Additional context

n/a

[tejasbubane]

There are two parts to this:

1. Add config option AllowComments - done in #7060
2. Do not allow rescue blocks consisting of nil (especially single line rescue) - I will do this once above is merged to avoid conflicts.

[rmm5t]

I like this, and I think it should be the new default.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution and understanding!

[tejasbubane]

Fixed in #7297

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution and understanding!

[sergeyk]

This issue should be closed, as it was fixed

[koic]

I agree with @rmm5t. I opened PR #7805.