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

📎 Support no-unescaped-entities rule #4491

Open
noahlt opened this issue Nov 8, 2024 · 3 comments
Open

📎 Support no-unescaped-entities rule #4491

noahlt opened this issue Nov 8, 2024 · 3 comments
Assignees
Labels
A-Linter Area: linter D-React Domain: React good first issue Good for newcomers L-JavaScript Language: JavaScript and super languages

Comments

@noahlt
Copy link

noahlt commented Nov 8, 2024

Description

Support eslint's no-unescaped-entities rule.

I'm open to taking a crack at this. I asked about this in the Linter rules from other sources thread, and @dyc3 instructed me to create this issue for further discussion.

@dyc3
Copy link
Contributor

dyc3 commented Nov 8, 2024

I've gone ahead and assigned you the task. The eslint rule has a forbid option, but for the initial implementation, don't add any options.

See CONTRIBUTING.md in biome_analyze for how to get started.

@lamualfa
Copy link

need this to align biome with current next.js default eslint configuration.

30:78  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities
38:54  Error: `"` can be escaped with `"`, `“`, `"`, `”`.  react/no-unescaped-entities
38:67  Error: `"` can be escaped with `"`, `“`, `"`, `”`.  react/no-unescaped-entities
53:9  Error: `"` can be escaped with `"`, `“`, `"`, `”`.  react/no-unescaped-entities
53:32  Error: `"` can be escaped with `"`, `“`, `"`, `”`.  react/no-unescaped-entities
60:27  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities
63:14  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities
89:43  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities
92:37  Error: `"` can be escaped with `"`, `“`, `"`, `”`.  react/no-unescaped-entities
92:53  Error: `"` can be escaped with `"`, `“`, `"`, `”`.  react/no-unescaped-entities
98:23  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities
107:77  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities

@dyc3 dyc3 added A-Linter Area: linter L-JavaScript Language: JavaScript and super languages D-React Domain: React good first issue Good for newcomers and removed A-Linter Area: linter L-JavaScript Language: JavaScript and super languages D-React Domain: React labels Mar 11, 2025
@dyc3
Copy link
Contributor

dyc3 commented Mar 11, 2025

@noahlt Are you still interested in taking this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Linter Area: linter D-React Domain: React good first issue Good for newcomers L-JavaScript Language: JavaScript and super languages
Projects
None yet
Development

No branches or pull requests

3 participants