-
Notifications
You must be signed in to change notification settings - Fork 8
Open
Description
To follow best practices, better ensure we're following the rules of react, and prepare our codebase for React Compiler, we should follow the react.dev documentation recommendations.
- https://react.dev/reference/react/StrictMode
- This was added in most (all?) of the MFEs during the React 18 upgrade (see https://github.com/search?q=org%3Aopenedx%20strictmode&type=code)
- https://www.npmjs.com/package/eslint-plugin-react-hooks
- This is at least installed in quite a few repos (https://github.com/search?q=org%3Aopenedx+eslint-plugin-react-hooks&type=code)
Current state of frontend-base
<StrictMode>
✅ Being used!
Line 26 in 0c48661
| <StrictMode> |
eslint-plugin-react-hooks
✅ Installed
Line 76 in 0c48661
| "eslint-plugin-react-hooks": "^5.0.0", |
🤷 Being used?
Metadata
Metadata
Assignees
Labels
No labels