-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Double quotes are enforced #10
Comments
This was discussed in standard/eslint-config-standard#27 a little while ago. You can take a look at the discussion there and see that we decided to go with double quotes. But standard v8 is still in beta until September 1, 2016 so this might change. Apologies for not marking the change as beta on this repo as well. |
Seemingly, the single quotes were chosen in eslint-config-standard-jsx :( The JSX is written like XML / HTML, not like JS. The single quote rule for the JSX, it's not productive :( |
@Nicolab this was no small decision. |
I imagine, in this case the easiest way is the more appropriate. Now that is done, I override this strange rule and it's no use that I argue more around the single quotes in JSX :) Sorry for my bad english |
It seems like the
jsx-quotes
rule was set to enforce double quotes (f85f775).Shouldn't we be using single quotes? I interpret JSX as JS so I'd use single quotes there too.
The text was updated successfully, but these errors were encountered: