rusackas opened a new pull request, #36413: URL: https://github.com/apache/superset/pull/36413
## Summary This PR removes `geostyler-qgis-parser` from direct dependencies as it's never used directly in the codebase and is already included as a transitive dependency through `geostyler`. ## Analysis After investigating the codebase: - **No direct imports**: Zero imports of `geostyler-qgis-parser` found anywhere in the codebase - **Already a sub-dependency**: The `geostyler` package (^14.1.3) already includes `geostyler-qgis-parser` (^2.0.0) as a dependency - **Redundant specification**: Having it as both a direct and transitive dependency is unnecessary ## Changes - Removed `geostyler-qgis-parser` (2.0.1) from package.json direct dependencies - Updated package-lock.json accordingly - The package remains available through `geostyler`'s dependencies ## Testing - Run `npm install` to verify dependencies resolve correctly - Verify map/geostyler functionality still works (the package is still available as a transitive dependency) - No functionality changes expected as the package was never directly imported ## Benefits - Cleaner dependency tree - Reduces maintenance burden (one less direct dependency to track) - Lets npm/yarn handle version resolution through the parent package - Follows best practices of not declaring unused direct 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]
