-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add sorting rules #39
base: main
Are you sure you want to change the base?
Conversation
react.js
Outdated
'react/jsx-boolean-value': 'error', | ||
'react/jsx-key': 'error', | ||
'react/jsx-no-duplicate-props': 'error', | ||
'react/jsx-no-undef': 'error', | ||
'react/jsx-sort-props': 'off', | ||
'react/jsx-sort-default-props': 'error', | ||
'react/jsx-sort-props': 'error', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See also db7a16d. Only pointing out that there are different opinions on this one. I don't have any strongly held opinions on these things as long as they are auto-fixable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just think the more rules we have the less the codebases can differ and the easier it is to switch between them going forward in Brian's and Quinn's idea to have teams less siloed in distinct projects 🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also understand there are different opinions on this one, at first I kind of disliked the sorting (as overkill), but now that I have gotten used to it it helps to find stuff more easily and read code better
49d17d3
to
125df47
Compare
No description provided.