Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fix module namespace #190

Merged
merged 1 commit into from
Jan 21, 2024
Merged

Conversation

cjrh
Copy link
Collaborator

@cjrh cjrh commented Jan 20, 2024

As identified by @pawamoy, the canonical import path is tantivy.tantivy. Our examples work with a single import tantivy only because the toplevel module re-imports everything. The correct hierarchy is required for documentation tools to real docstrings correctly.

As identified by @pawamoy, the canonical import
path is `tantivy.tantivy`. Our examples work with
a single `import tantivy` only because the
toplevel module re-imports everything. The correct
hierarchy is required for documentation tools
to real docstrings correctly.
@cjrh cjrh merged commit cde36c2 into quickwit-oss:master Jan 21, 2024
10 checks passed
Sidhant29 pushed a commit to Sidhant29/tantivy-py that referenced this pull request Mar 15, 2024
…t-oss#190)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@cjrh cjrh deleted the fix-module-hierarchy branch May 3, 2024 20:24
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.

2 participants