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

Fix JSX Syntax #1867

Merged
merged 1 commit into from Jun 30, 2018
Merged

Fix JSX Syntax #1867

merged 1 commit into from Jun 30, 2018

Conversation

peter-mouland
Copy link
Contributor

No description provided.

@brokentone
Copy link

This looks like an improvement to me!

@ljharb
Copy link
Member

ljharb commented Jun 30, 2018

Why did you close #1866 and open this, instead of updating that PR?

@ljharb ljharb merged commit c3e57c7 into jsx-eslint:master Jun 30, 2018
@peter-mouland
Copy link
Contributor Author

There are 2 mistakes in the original PR!
Thanks for merging

@peter-mouland peter-mouland deleted the patch-4 branch June 30, 2018 09:28
@ljharb
Copy link
Member

ljharb commented Jun 30, 2018

@peter-mouland in the future, please update the original PR instead of replacing it. I’ll be happy to walk you through it if you have trouble.

This was referenced Sep 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants