Feature: no-inline-comments: Allow exception inside JSX

[alexilyaev]

What rule do you want to change?
no-inline-comments
Does this change cause the rule to produce more or fewer warnings?
Fewer
How will the change be implemented? (New option, new default behavior, etc.)?
Rule options, or change in rule behavior
Please provide some example code that this change will affect:

function App() {
  return (
    <div>
      {/* Some comment */}
      <h1>Some heading</div>
    </div>
  )
}

What does the rule currently do for this code?
Right now no-inline-comments will warn on any comments inside JSX.
What will the rule do after it's changed?
It makes sense to allow comments inside JSX while still disallowing inline comments in regular JS blocks.
Are you willing to submit a pull request to implement this change?
Unfortunately not.

[kaicataldo]

Since comments require the {/} brackets (and I'm assuming that's what triggering the rule to fire), this does seem like a reasonable behavior to add in with an option. I think it would have to ensure that the brackets only contain a comment and treat the brackets as the delimiters to check for tokens around the comment rather than the comment itself.

[eslint-deprecated]

Unfortunately, it looks like there wasn't enough interest from the team
or community to implement this change. While we wish we'd be able to
accommodate everyone's requests, we do need to prioritize. We've found
that issues failing to reach accepted status after 21 days tend to
never be accepted, and as such, we close those issues.
This doesn't mean the idea isn't interesting or useful, just that it's
not something the team can commit to.

Thanks for contributing to ESLint and we appreciate your understanding.

[platinumazure]

Reopening. I'll champion this.

[g-plane]

I'd like to add an option called "ignoreJSX".

[kaicataldo]

@platinumazure @g-plane It sounds like we have two different implementation ideas here (assuming @platinumazure was championing something in neighborhood of what I was suggesting). How should we proceed?

[platinumazure]

I don't feel strongly either way. I would be okay with an option, but I also note that this rule is pretty much unusable in JSX without the new option and I think we could easily avoid a breaking change in non-JSX code even if we change the default behavior.

[kaicataldo]

The current behavior in JSX does feel like a bug to me, and I would be okay with changing the behavior by default for JSX as long as other cases aren't affected. If we make this change, I do think we should note the reasoning for this exception in the docs. It would be even better if we could get this into v6!

[kaicataldo]

@eslint/eslint-team Looks like we need one more 👍 or it might be time to close this issue. @platinumazure are you still championing this change?

[jumpconnector]

+1 on this. There should be an option to allow exception on comments inside JSX

[platinumazure]

@kaicataldo Yes, I'll champion still. I'll poke in the team chat for another 👍 from the team 😄

[dorian-marchal]

Looks like we need one more 👍

+1 👍

[platinumazure]

Thanks @dorian-marchal... We're looking for a 👍 from a member of the ESLint team. (Please review our consensus process for more context.)

That said, we did get another team member's 👍, so this issue is accepted.

[mdjermanovic]

I'm working on this.