-
-
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
Fix false negative for Rails/LinkToBlank when _blank is a symbol #6821
Conversation
d9351f8
to
7b26e6a
Compare
CHANGELOG.md
Outdated
@@ -4,6 +4,7 @@ | |||
|
|||
### Bug fixes | |||
|
|||
* [#6821](https://github.com/rubocop-hq/rubocop/pull/6821): Fix false positive for Rails/LinkToBlank when _blank is a symbol. ([@Intrepidd][]) |
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.
What is not able to find a problem is "false negative" (not "false positive") :-)
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.
Ok I thought as a false positive as in "your code is ok" when it is not.
I'll update my PR right away
Thanks !
7b26e6a
to
f27d79f
Compare
CHANGELOG.md
Outdated
@@ -4,6 +4,7 @@ | |||
|
|||
### Bug fixes | |||
|
|||
* [#6821](https://github.com/rubocop-hq/rubocop/pull/6821): Fix false negative for Rails/LinkToBlank when _blank is a symbol. ([@Intrepidd][]) |
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.
This is a nitpick. It seems that _
intents italic start if _blank
is not enclosed in backticks.
-* [#6821](https://github.com/rubocop-hq/rubocop/pull/6821): Fix false negative for Rails/LinkToBlank when _blank is a symbol. ([@Intrepidd][])
+* [#6821](https://github.com/rubocop-hq/rubocop/pull/6821): Fix false negative for Rails/LinkToBlank when `_blank` is a symbol. ([@Intrepidd][])
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.
Thanks, fixed
f27d79f
to
53597d0
Compare
Thanks! |
when using
target: :_blank
, theRails/LinkToBlank
cop does not register an offence when it should.