Skip to content
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

Add missing React-related eslint rules #26

Merged
merged 4 commits into from Jul 19, 2021
Merged

Conversation

odididi
Copy link
Contributor

@odididi odididi commented Jul 16, 2021

No description provided.

@odididi odididi requested a review from a team July 16, 2021 14:29
@odididi odididi force-pushed the odididi/update-react-rules branch from 4b1a3dc to 217faad Compare July 16, 2021 14:30
@Avaq Avaq assigned Avaq and d-eisenga and unassigned Avaq Jul 19, 2021
jsx.js Outdated Show resolved Hide resolved
jsx.js Outdated Show resolved Hide resolved
jsx.js Show resolved Hide resolved
@Avaq
Copy link
Contributor

Avaq commented Jul 19, 2021

Before we merge #26 and #25 into main, I'll make a temporary next-branch and merge them both there, so we can try it out with npm i wearereasonablepeople/eslint-config-warp#next.

Avaq added 2 commits July 19, 2021 16:47
This rule, when enabled, prevents using React's string interpolation,
for example:

    <a>
      Hello {thing}!
    </a>

Would have to be changed to

    <a>
      {`Hello ${thing}!`}
    </a>

And while the rule itself is useful, its primary use is already covered
by jsx-closing-bracket-location.

This commit disables the rule to allow string interpolation, and we'll
rely on jsx-closing-bracket-location to prevent multiple opening or
closing tags on the same line.
@Avaq Avaq merged commit 10a2ac8 into main Jul 19, 2021
@Avaq Avaq deleted the odididi/update-react-rules branch July 19, 2021 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants