JasjotBenipal commented on issue #45258: URL: https://github.com/apache/superset/issues/45258#issuecomment-6256500042
We ran the frontend linter using npm run lint with the rule configured as an error. This identified 54 violations across multiple files and modules that need to be addressed before the rule can be fully enforced. To keep the changes focused and make reviews more manageable, we plan to split the refactoring into four PRs grouped by related functionality: PR 1: Chart-related modules PR 2: Shared UI components PR 3: Explore controls and map components PR 4: Dashboard and leftover components For each group, we will review the affected imports and their default and named exports and refactor the imports or exports to resolve each violation without changing existing behavior. We will re-run linting and relevant tests to validate the changes. Once all violations are resolved and the linter passes, we'll ensure import/no-named-as-default is configured as an error to prevent future violations. This step by step approach will keep each PR focused and simplify code review. -- 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]
