-
Notifications
You must be signed in to change notification settings - Fork 46
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
front: add eslint rule to ban React.FC #8664
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #8664 +/- ##
============================================
- Coverage 36.87% 36.83% -0.05%
- Complexity 2177 2217 +40
============================================
Files 1255 1255
Lines 113719 113832 +113
Branches 3203 3187 -16
============================================
- Hits 41938 41931 -7
- Misses 69886 70009 +123
+ Partials 1895 1892 -3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Nice!
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.
Thank you for this PR ! The eslint config doesn't catch type FC
in import React
(FC is still there in all IncompatibleConstraints components)
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.
Lgtm
Co-authored-by: Simon Ser <[email protected]>
7e37183
to
9655fc9
Compare
Indeed! Added a rule for lone |
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.
Lgtm, thank you for the follow up @emersion :)
closes #8663