Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upBetter JSX handling #415
Comments
This comment has been minimized.
This comment has been minimized.
|
Oh yeah, that rule seems reasonable, PR welcome! Regarding JSX - I think the reason it's less thorough might be because there've been less contributions to JSX compared to other rules. We don't particularly have a policy regarding what is / isn't acceptable as a rule - it's determined on a case-by-case basis. If you run into anything that you think should be part of |
This comment has been minimized.
This comment has been minimized.
|
That's great to hear thanks for the quick reply, I'll open a PR at https://github.com/feross/eslint-config-standard-jsx |
nickcolley
closed this
Feb 10, 2016
This comment has been minimized.
This comment has been minimized.
|
The "react/jsx-curly-spacing": [2, "never"] |
lock
bot
locked as resolved and limited conversation to collaborators
May 10, 2018
nickcolley commentedFeb 10, 2016
I love Standard but it falls down with JSX, is the project looking to be transparently opinionated like it is with the rest of JS, or is this something that'd need to be in a fork?
Things like https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-curly-spacing.md for example is a pain to not have consistent.