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

fix(jsx-curly-brace-presence): allow necessary white-space literal #2437

Merged
merged 2 commits into from Oct 3, 2019

Conversation

uniqname
Copy link

@uniqname uniqname commented Oct 2, 2019

Checks against use-case where white space literal is used to pad around JSX across newlines.

@ljharb ljharb force-pushed the fix/jsx-curly-brace-presence branch from d1ec25f to 4c89872 Compare October 3, 2019 18:11
@ljharb ljharb force-pushed the fix/jsx-curly-brace-presence branch from 4c89872 to 156e9f5 Compare October 3, 2019 18:20
@ljharb ljharb force-pushed the fix/jsx-curly-brace-presence branch 3 times, most recently from 023be90 to 6f14e16 Compare October 3, 2019 18:58
@ljharb ljharb merged commit 6f14e16 into jsx-eslint:master Oct 3, 2019
eventualbuddha added a commit to votingworks/bmd that referenced this pull request Oct 7, 2019
This was incorrectly removed by eslint-plugin-react v7.15.x due to a bug. It's been fixed in jsx-eslint/eslint-plugin-react#2437, so this commit updates eslint-plugin-react and re-adds the whitespace.
eventualbuddha added a commit to votingworks/bmd that referenced this pull request Oct 7, 2019
This was incorrectly removed by eslint-plugin-react v7.15.x due to a bug. It's been fixed in jsx-eslint/eslint-plugin-react#2437, so this commit updates eslint-plugin-react and re-adds the whitespace.
beausmith pushed a commit to votingworks/vxsuite that referenced this pull request Nov 2, 2020
This was incorrectly removed by eslint-plugin-react v7.15.x due to a bug. It's been fixed in jsx-eslint/eslint-plugin-react#2437, so this commit updates eslint-plugin-react and re-adds the whitespace.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants