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: do not trigger on strings containing a quote character #3798

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

akulsr0
Copy link
Contributor

@akulsr0 akulsr0 commented Aug 8, 2024

Fixes #3795

@ljharb ljharb changed the title [Fix] jsx-curly-brace-presence: doesn't trigger on strings containing a quote character [Fix] jsx-curly-brace-presence: do not trigger on strings containing a quote character Aug 9, 2024
@ljharb ljharb merged commit 10eb235 into jsx-eslint:master Aug 9, 2024
343 checks passed
renovate bot added a commit to mmkal/eslint-plugin-mmkal that referenced this pull request Sep 3, 2024
@dennisoelkers
Copy link

dennisoelkers commented Sep 3, 2024

@akulsr0, @ljharb: This PR is introducing a regression for us, it is changing:

help={'The maximum time range for searches. (i.e. "P30D" for 30 days, "PT24H" for 24 hours)'}

to

help="The maximum time range for searches. (i.e. "P30D" for 30 days, "PT24H" for 24 hours)"

which is obviously an incorrect string.

@ljharb
Copy link
Member

ljharb commented Sep 3, 2024

@dennisoelkers you’re right; could you file a new issue for that?

@dennisoelkers
Copy link

@ljharb: There you go: #3814

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.

[Bug]: jsx-curly-brace-presence doesn't trigger on strings containing a quote character
3 participants