Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork3.4k
Open
Description
This is a little-known (I believe) and rarely used feature of the rescue operator, which allows assigning an exception using an object's writer method:
Foo=Struct.new(:exception)foo=Foo.newbegindo_something_that_might_raiserescue=>foo.exception# `Foo#exception=` will be calledRails.error.report(foo.exception)do_something_with_exception(foo.exception)end
I propose introducing a new cop that detects such patterns and suggests expanding them to
Foo=Struct.new(:exception)foo=Foo.newbegindo_something_that_might_raiserescue=>efoo.exception=e# if requiredRails.error.report(e)do_something_with_exception(e)end
This cop could also support an alternative style that, conversely, suggests using the writer variant (where applicable).
Metadata
Metadata
Assignees
Labels
No labels