New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Lint/RaiseException rule #8

Merged

olleolleolle merged 1 commit into master from olleolleolle-add-rule-lint-raise-exception

Sep 14, 2020

Contributor

olleolleolle commented Sep 14, 2020

introduced in 0.81.0, [Fix #7299] Implement Lint/RaiseException cop rubocop/rubocop#7325

Short story:

# bad
raise Exception, 'Error message here'
# good
raise StandardError, 'Error message here'


          Add Lint/RaiseException rule

36e4174

  - introduced in 0.81.0, rubocop/rubocop#7325

Member

henrik commented Sep 14, 2020

Sure! Think this one is uncontroversial but might catch mistakes. I assume raise "hi" is also "good" :)

henrik self-requested a review

September 14, 2020 09:01

henrik approved these changes

View reviewed changes

olleolleolle merged commit 9f41d0d into master

olleolleolle deleted the olleolleolle-add-rule-lint-raise-exception branch

September 14, 2020 09:36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment