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

feat: add py.typed to allow mypy to type check packages that use ibis #5304

Merged
merged 1 commit into from
Jan 23, 2023

Conversation

tswast
Copy link
Collaborator

@tswast tswast commented Jan 23, 2023

In checking a package I am working on, I noticed the BaseBackend.table() method was missing, and it appears this method should be available for all backends, so I added an abc method for it.

Note: Since Ibis is using poetry, I don't think anything else is needed to include this file. See: https://jugmac00.github.io/blog/bite-my-shiny-type-annotated-library/#poetry

Fixes #5279

In checking a package I am working on, I noticed the BaseBackend.table()
method was missing, and it appears this method should be available for
all backends, so I added an abc method for it.
@github-actions
Copy link
Contributor

github-actions bot commented Jan 23, 2023

Test Results

       43 files         43 suites   1h 47m 15s ⏱️
13 792 tests 10 309 ✔️   3 483 💤 0
47 711 runs  35 164 ✔️ 12 547 💤 0

Results for commit 2116b09.

♻️ This comment has been updated with latest results.

@cpcloud cpcloud added this to the 4.1 milestone Jan 23, 2023
@cpcloud cpcloud added feature Features or general enhancements developer-tools Tools related to ibis development labels Jan 23, 2023
Copy link
Member

@cpcloud cpcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet, thanks!

@cpcloud cpcloud merged commit 765d42e into ibis-project:master Jan 23, 2023
@tswast tswast deleted the issue5279-py.typed branch January 23, 2023 16:42
@binste binste mentioned this pull request Aug 12, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer-tools Tools related to ibis development feature Features or general enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: add py.typed marker to allow packages using ibis to benefit from ibis's type annotations
2 participants