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

New rule: jsx-no-trailing-semicolon #116

Closed
jkillian opened this issue Aug 28, 2017 · 2 comments
Closed

New rule: jsx-no-trailing-semicolon #116

jkillian opened this issue Aug 28, 2017 · 2 comments

Comments

@jkillian
Copy link
Contributor

jkillian commented Aug 28, 2017

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>

(haven't thought about the rule name that much, so take the title with a grain of salt)

@tomauty
Copy link

tomauty commented Aug 14, 2018

I was just looking for something like this. Maybe no-errant-characters? Is there an actual use-case for having characters not wrapped in an element in a JSX statement? For RN there is not.

@adidahiya
Copy link
Contributor

Closing due to deprecation timeline #210, let's focus on getting support in eslint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants