[internal] Fix eslint-plugin-react-compiler issues
Open
@aarongarciah is already working on this.
Since Jun 7, 2024.
internal
ready to take
- Dominant language
- JavaScript
- Stars
- 99.1k
- Forks
- 32.5k
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 106
Description
Part of https://github.com/mui/material-ui/issues/42548
As part of adopting the official ESLint plugin for the React Compiler, we want to solve all of the reported issues.
Contributing
- Pick a file (or group of files belonging to the same group, ideally) in the list of existing issues below and let us know in the comments so we can assign them to you.
- Enable
eslint-plugin-react-compilerin your local setup.
The plugin is disabled by default, you won't see issues reported by it unless you enable it manually. To enable it, go to the root.eslintrc.jsfile and setENABLE_REACT_COMPILER_PLUGINtotrue. You should start seeing the issues reported right into your IDE if it's configured to report ESLint errors. - Once the issue is solved, open a PR mentioning this issue so we can link it in the description and keep it up to date.
When in doubt if an issue can be solved or not (because of false positives or any other reason), explain why in your PR so we can discuss if the ESLint should be silenced in that case.
Issues to fix
Here's the list of files containing issues reported by eslint-plugin-react-compiler (as of June 6th 2024).
mui-material
Demos
- docs/data/material/components/steppers/HorizontalNonLinearStepper.tsx (#42559)
- docs/data/material/components/use-media-query/UseWidth.tsx
Components / hooks (#43117, #43946)
- packages/mui-material/src/Backdrop/Backdrop.js
- packages/mui-material/src/ButtonBase/ButtonBase.js
- packages/mui-material/src/FormControl/FormControl.js
- packages/mui-material/src/Hidden/withWidth.js
- packages/mui-material/src/InputBase/InputBase.js
- packages/mui-material/src/InputBase/InputBase.test.js
- packages/mui-material/src/Popover/Popover.js
- packages/mui-material/src/Select/SelectInput.js
- packages/mui-material/src/Snackbar/Snackbar.test.js
- packages/mui-material/src/SwipeableDrawer/SwipeableDrawer.js
- packages/mui-material/src/Tooltip/Tooltip.js
- packages/mui-material/src/styles/useTheme.js
- packages/mui-material/src/usePagination/usePagination.test.js
- packages/mui-material/src/useScrollTrigger/useScrollTrigger.js
mui-lab (#42880)
- packages/mui-lab/src/Alert/Alert.js
- packages/mui-lab/src/AlertTitle/AlertTitle.js
- packages/mui-lab/src/Autocomplete/Autocomplete.js
- packages/mui-lab/src/AvatarGroup/AvatarGroup.js
- packages/mui-lab/src/Pagination/Pagination.js
- packages/mui-lab/src/PaginationItem/PaginationItem.js
- packages/mui-lab/src/Rating/Rating.js
- packages/mui-lab/src/Skeleton/Skeleton.js
- packages/mui-lab/src/SpeedDial/SpeedDial.js
- packages/mui-lab/src/SpeedDialAction/SpeedDialAction.js
- packages/mui-lab/src/SpeedDialIcon/SpeedDialIcon.js
- packages/mui-lab/src/TabContext/TabContext.test.js
- packages/mui-lab/src/ToggleButton/ToggleButton.js
- packages/mui-lab/src/ToggleButtonGroup/ToggleButtonGroup.js
mui-styles (#43118)
- packages/mui-styles/src/StylesProvider/StylesProvider.js
- packages/mui-styles/src/makeStyles/makeStyles.js
- packages/mui-styles/src/makeStyles/makeStyles.spec.tsx
- packages/mui-styles/src/withStyles/withStyles.js
mui-system (#42637)
- packages/mui-system/src/ThemeProvider/ThemeProvider.test.js
mui-docs
- packages/mui-docs/src/CodeCopy/CodeCopy.tsx
- packages/mui-docs/src/i18n/i18n.tsx (#42881)
mui-utils (#43051)
- packages/mui-utils/src/useControlled/useControlled.js
- packages/mui-utils/src/useForkRef/useForkRef.ts
- packages/mui-utils/src/useId/useId.ts
- packages/mui-utils/src/useIsFocusVisible/useIsFocusVisible.ts
- packages/mui-utils/src/useOnMount/useOnMount.ts
mui-joy (#42671)
- packages/mui-joy/src/Checkbox/Checkbox.tsx
- packages/mui-joy/src/FormControl/FormControl.tsx
- packages/mui-joy/src/Input/Input.tsx
- packages/mui-joy/src/Radio/Radio.tsx
- packages/mui-joy/src/RadioGroup/RadioGroup.tsx
- packages/mui-joy/src/Select/Select.tsx
- packages/mui-joy/src/Switch/Switch.tsx
- packages/mui-joy/src/Textarea/Textarea.tsx
- packages/mui-joy/src/Tooltip/Tooltip.tsx
- packages/mui-joy/src/styles/ThemeProvider.tsx
- packages/mui-joy/src/styles/extendTheme.test.js
Test (#42626)
- packages-internal/test-utils/src/mochaHooks.test.js
- test/regressions/TestViewer.js
mui-private-theming (#42619)
- packages/mui-private-theming/src/ThemeProvider/ThemeProvider.js
- packages/mui-private-theming/src/useTheme/useTheme.js
Documentation website components (#42566)
- docs/src/components/banner/AppFrameBanner.tsx
- docs/src/components/productMaterial/MaterialStyling.tsx
- docs/src/components/productX/XHero.tsx
- docs/src/modules/components/ApiPage/sections/ToggleDisplayOption.tsx
- docs/src/modules/components/AppNavDrawer.js
- docs/src/modules/components/DemoSandbox.js
- docs/src/modules/components/DemoToolbar.js
- docs/src/modules/components/ReactRunner.tsx
- docs/src/modules/components/ThemeContext.js
Search keywords: ESLint, React Compiler
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.