Fix: Add explicit re-exports for public API#1143
Fix: Add explicit re-exports for public API#1143anaslimem wants to merge 1 commit intoqdrant:masterfrom
Conversation
- Add explicit re-export of BearerAuth in auth/__init__.py - Add explicit re-export of migrate in migrate/__init__.py This fixes F401 linting warnings and makes the public API clearer for type checkers (mypy, pyright).
✅ Deploy Preview for poetic-froyo-8baba7 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThe pull request updates import statements in two Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
This PR adds explicit re-exports to public API modules to fix linting warnings and improve type checking clarity.
Changes Made
BearerAuthimport to explicit re-export syntaxmigrateimport to explicit re-export syntaxMotivation
When using
from module import namein init.py, type checkers (mypy, pyright) can't determine if the import is for internal use or part of the public API. Usingimport name as nameexplicitly signals that this is a public re-export.This pattern:
Testing
ruff checkconfirms linting issues are resolvedType of Change