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
[New] jsx-no-target-blank
: add fixer
#2862
Conversation
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.
Almost there :-)
errors: defaultErrors | ||
}, { | ||
code: '<a target={"_blank"} href={"//example.com"} rel={"noopenernoreferrernoreferrernoreferrernoreferrernoreferrer"}></a>', | ||
output: '<a target={"_blank"} href={"//example.com"} rel={"noopener noreferrer"}></a>', |
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.
it's fine to keep it here since the original was both a string and had curly braces
jsx-no-target-blank
: add fixer
de33e0b
to
907c8e8
Compare
907c8e8
to
11eaaae
Compare
@Nokel81 ok! i think this time i've got it down to 4 failures, not just in older eslint versions: namely, that the fixer is fixing the code in the spread attribute case. If CI ends up showing those same 4 failures, then fixing that would be the last step to landing this. |
Please mark the PR as ready for review once either the tests are passing or you're stuck again (hopefully the former :-p ) |
b914d29
to
26a9307
Compare
@ljharb All tests are passing. |
26a9307
to
4f2d1eb
Compare
No description provided.