feat: add skipJSXText option to no-irregular-whitespace rule

[azat-io]

Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request? (put an "X" next to an item)

• Documentation update
• Bug fix (template)
• New rule (template)
• Changes an existing rule (template)
• Add autofix to a rule
• Add a CLI option
• Add something to the core
• Other, please explain:

What changes did you make? (Give an overview)

Currently you can use irregular whitespace with rule options in strings: https://eslint.org/docs/latest/rules/no-irregular-whitespace#options
Sometimes I use non-breaking spaces also in JSX text. It would be nice to have this option in no-irregular-whitespace rule.

Is there anything you'd like reviewers to focus on?

Nope

What rule do you want to change?

no-irregular-whitespace

What change do you want to make (place an "X" next to just one item)?

• Generate more warnings
• Generate fewer warnings
• Implement autofix
• Implement suggestions

None of this. I added one more option to rule.

How will the change be implemented (place an "X" next to just one item)?

• A new option
• A new default behavior
• Other

Please provide some example code that this change will affect:

Currently I cannot do that:

<div>Some text with irregular spaces</div>

What does the rule currently do for this code?

Currently, if I use non-breaking spaces in JSX text, I get the error.

What will the rule do after it's changed?

Added option to use irregular spaces in JSX text, similar to options for regular expressions and strings.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: azat-io / name: Azat S. (b3dddfd, 59390be, 897b358, 1424b3f, 95c6095, b1f0db4)

[netlify]

✅ Deploy Preview for docs-eslint canceled.

Name	Link
🔨 Latest commit	877ad2d
🔍 Latest deploy log	https://app.netlify.com/sites/docs-eslint/deploys/647a3e5eaec3b20008f659b3

[Rec0iL99]

Hi @azat-io, thanks for the suggestion. Please follow the pull request template in the description providing more info for the reviewers. Thanks.

[azat-io]

@Rec0iL99 Done ✅

[mdjermanovic]

@azat-io can you also copy & paste this template into the original post and fill it out?

[azat-io]

@mdjermanovic Done ✅

[mdjermanovic]

Thanks for the input! Given that this rule already has an option to ignore strings, I think it makes sense to add an option to ignore JSX text too, so I support this proposal 👍

Let's hear what other team members think about the proposal before reviewing the code.

[azat-io]

Any feedback?
Sorry for the ping, I'm looking forward to this feature.

[azat-io]

Ping

[Rec0iL99]

@azat-io some of our team members are unavailable at the moment. Please be patient as we might experience some delays in the review process.

[nzakas]

I'm support this change. 👍

[nzakas]

LGTM. Would like another team member to verify before merging.

[mdjermanovic]

Code LGTM, I left suggestions about the docs and tests.

[mdjermanovic]

@azat-io can you fix lint errors (trailing commas in invalid tests)?

Otherwise, this looks good, but since it includes a documentation update, due to #17225 this will have to wait for #17231 to be merged first, and then this branch should be rebased on top of main.

[azat-io]

@mdjermanovic Done ✅

Strange, lint command does not work for me:

[mdjermanovic]

@azat-io looks like your package manager doesn't work with file: links. Either way, the lint errors are fixed now, and everything looks good, but since this includes a documentation update and there were certain licensing issues (#17225) that are now resolved on the main branch, could you please rebase this branch on top of main?

[azat-io]

@mdjermanovic Rebased ✅

[mdjermanovic]

LGTM, thanks!