Skip to content

Fix: Add explicit re-exports for public API#1143

Open
anaslimem wants to merge 1 commit intoqdrant:masterfrom
anaslimem:fix/add-explicit-reexports
Open

Fix: Add explicit re-exports for public API#1143
anaslimem wants to merge 1 commit intoqdrant:masterfrom
anaslimem:fix/add-explicit-reexports

Conversation

@anaslimem
Copy link
Copy Markdown

Description

This PR adds explicit re-exports to public API modules to fix linting warnings and improve type checking clarity.

Changes Made

  • auth/init.py: Change BearerAuth import to explicit re-export syntax
  • migrate/init.py: Change migrate import to explicit re-export syntax

Motivation

When using from module import name in init.py, type checkers (mypy, pyright) can't determine if the import is for internal use or part of the public API. Using import name as name explicitly signals that this is a public re-export.

This pattern:

  • Fixes F401 linting warnings from ruff
  • Improves API clarity for type checkers
  • Follows PEP 484 type hinting standards
  • Makes it clear these are intentional public exports

Testing

  • ruff check confirms linting issues are resolved
  • No functional changes - purely type hint improvements
  • All existing imports continue to work

Type of Change

  • Code quality improvement
  • Bug fix
  • New feature
  • Breaking change

- 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).
@netlify
Copy link
Copy Markdown

netlify bot commented Jan 18, 2026

Deploy Preview for poetic-froyo-8baba7 ready!

Name Link
🔨 Latest commit 00c3a45
🔍 Latest deploy log https://app.netlify.com/projects/poetic-froyo-8baba7/deploys/696cf5d1e1d01600083baf62
😎 Deploy Preview https://deploy-preview-1143--poetic-froyo-8baba7.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 18, 2026

📝 Walkthrough

Walkthrough

The pull request updates import statements in two __init__.py files within the qdrant_client package. In qdrant_client/auth/__init__.py, the BearerAuth import is restructured while preserving its public export. Similarly, in qdrant_client/migrate/__init__.py, the migrate import is updated without altering the public API. Both changes maintain existing functionality and expose the same symbols to consumers of the library.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main purpose of the changeset: adding explicit re-exports for public API modules to fix linting warnings.
Description check ✅ Passed The description is directly related to the changeset, explaining the motivation, specific changes made, and benefits of the re-export pattern improvements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant