rusackas opened a new pull request, #36433: URL: https://github.com/apache/superset/pull/36433
## Summary This PR removes the deprecated `react-hot-loader` package since React Refresh is already configured and working in the project via the SWC loader. ### Changes: - Removed `react-hot-loader` dependency from package.json - Removed hot() wrapper from App.tsx - Removed setHotLoaderConfig from preamble.ts - Removed react-hot-loader/babel plugin from babel.config.js - Removed react-hot-loader from webpack vendor bundle - Cleaned up related comments ## Before/After **Before:** Project had both react-hot-loader (deprecated) and React Refresh configured **After:** Only React Refresh is used for hot module replacement ## Testing Instructions 1. Run `npm run dev-server` 2. Make changes to React components 3. Verify hot module replacement works correctly (components update without full page refresh) 4. Verify no console errors related to hot module replacement ✅ **Tested locally:** Dev server compiles successfully and HMR works as expected. ## Additional Information React Refresh has been the recommended HMR solution for React since React 17. The react-hot-loader package is deprecated and no longer maintained. This migration simplifies our development setup by removing unnecessary dependencies. 🤖 Generated with [Claude Code](https://claude.ai/code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
