rusackas opened a new pull request, #36430: URL: https://github.com/apache/superset/pull/36430
## Summary This PR removes the `luxon` date/time library which is not used anywhere in the Superset frontend codebase. ## Analysis Using dependency analysis tooling, I verified that: - **Zero imports**: No imports of `luxon` found in any source files (src/, packages/, plugins/) - **No transitive dependencies**: Not required by any other packages - **No build/config usage**: Not referenced in webpack, babel, or other configuration files - **Not in scripts**: Not used in any npm scripts ## Changes - Removed `luxon` (^3.7.1) from package.json dependencies - Updated package-lock.json accordingly ## Impact - **Bundle size reduction**: Luxon is approximately 73KB minified - **Reduced maintenance**: One less dependency to track and update - **No functionality changes**: The library was completely unused ## Testing - Verified no imports exist: `rg "from ['\"]luxon|import.*luxon|require.*luxon"` - Frontend builds successfully - No TypeScript or linting errors ## Context Luxon appears to have been added but never used, possibly considered as an alternative to existing date libraries (dayjs, moment) but not adopted. The codebase primarily uses dayjs for date/time operations. 🤖 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]
