-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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.
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
[Fixes #10439] Add Style/RedundantStringEscape #11002
Conversation
module RuboCop | ||
module Cop | ||
module Style | ||
# Checks for redundant escapes in Strings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in string literals
include RangeHelp | ||
extend AutoCorrector | ||
|
||
MSG = 'Redundant escape inside string literal.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps we should mention the exact escape in the message?
module Style | ||
# Checks for redundant escapes in Strings. | ||
# | ||
# @example |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably you should add a couple of examples with heredocs as well, so it's clear to everyone those are considered.
Well done! |
CI seems to have failed because rubocop/rubocop#11002 was merged (e.g. rubocop@2723be4) This commit is for fixing it.
CI seems to have failed because rubocop/rubocop#11002 was merged (e.g. rubocop@2723be4) This commit is for fixing it.
CI seems to have failed because rubocop/rubocop#11002 was merged (e.g. rubocop@2723be4) This commit is for fixing it.
Add new
Style/RedundantStringEscape
cop, as suggested in #10439Before submitting the PR make sure the following are checked:
[Fix #issue-number]
(if the related issue exists).master
(if not - rebase it).bundle exec rake default
. It executes all tests and runs RuboCop on its own code.{change_type}_{change_description}.md
if the new code introduces user-observable changes. See changelog entry format for details.