villebro opened a new pull request, #43626: URL: https://github.com/apache/superset/pull/43626
### SUMMARY `superset_core` carried `__all__` lists in 15 modules, but nothing relies on the convention — there are no star-imports (`from superset_core... import *`) of these modules anywhere in the codebase (verified). The lists are therefore dead maintenance overhead that silently drifts out of sync as symbols are added (e.g. a recently-added model wasn't in its module's list). This removes `__all__` from all 15 modules. No public API change: without `__all__`, `import *` (which nothing uses here) would export the same non-underscore names; normal `from x import Name` imports are unaffected. ### TESTING INSTRUCTIONS `ruff check` / `ruff format --check` clean on the changed files; `grep -rn 'import \*' ` confirms no star-imports of these modules. Import surface is unchanged for explicit imports. ### ADDITIONAL INFORMATION - [ ] Has associated issue: - [ ] Changes UI - [ ] Includes DB Migration - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
