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(python): Disable type checking for dataframe_api_compat dependency #11997

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

stinodego
Copy link
Member

@stinodego stinodego commented Oct 24, 2023

This is the second time this package breaks our CI due to type hint issues.

I opened a PR in the repo to fix this specific one, but it seems that there is no type checking in their CI. So I think it's best to turn off type checking for this package until the type hints are more reliable.

@MarcoGorelli FYI - I don't mind helping out a bit in that repo with regards to type hints, with the Polars side of things at least. Let me know if there is something I can help you with.

@github-actions github-actions bot added internal An internal refactor or improvement python Related to Python Polars rust Related to Rust Polars labels Oct 24, 2023
@stinodego stinodego changed the title chore: Fix mypy error chore(python): Disable type checking for dataframe_api_compat dependency Oct 24, 2023
@stinodego stinodego merged commit 71d2156 into main Oct 24, 2023
13 checks passed
@stinodego stinodego deleted the fix-mypy-issue branch October 24, 2023 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal An internal refactor or improvement python Related to Python Polars rust Related to Rust Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant