Suggestion: no trailing semicolon

[OliverJAsh]

Copying suggestion from palantir/tslint-react#116:

We had a case in our code where a semicolon was accidentally used at the end of a line of a tsx statement. Since this is almost always unintended, it makes sense to have a lint rule to warn against it. The rule would warn against code like the following:

<div>
  <SomeComponent/>;
</div>

/cc @jkillian

[ljharb]

This seems like a reallllllly niche rule for something that almost never happens :-/

[OliverJAsh]

Happened to me loads of times. The linked issue also has some interest suggesting it's not just me.

[ljharb]

This seems like it would be handled by the jsx-no-literals rule in a much more generic way.

[OliverJAsh]

That seems quite extreme for what we're trying to achieve here. I'm fine with literals, just not trailing semicolons, which usually get left behind during refactors

[dgoldstein0]

+1 for this being useful

[bobthecow]

👍 I've fixed instances of this bug four or five times in production. I'd love a linter too!

[cjvnjde]

Not the best solution, but it works for me.

    "no-restricted-syntax": [
      "error",
      {
        selector: "JSXElement > JSXText[value=/^;\n/]",
        message: 'Please use proper case for the text node "semi" after JSX elements.',
      },
    ]

It works for a semicolon at the end of the line after a jsx tag.