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

[docs] Include Polars as supported Pandera backend #26596

Merged
merged 1 commit into from
Dec 24, 2024
Merged

Conversation

deepyaman
Copy link
Contributor

Summary & Motivation

#23299 introduced support for Polars, but the docs weren't updated accordingly (which I learned when trying to figure out whether there was a reason only pandas was supported).

If so desired, I can update to also include Polars examples.

@deepyaman deepyaman requested a review from neverett as a code owner December 19, 2024 16:17
@deepyaman deepyaman requested a review from smackesey December 19, 2024 16:24
@neverett neverett added the docs-to-migrate Docs to migrate to new docs site label Dec 20, 2024
Copy link
Contributor

@neverett neverett left a comment

Choose a reason for hiding this comment

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

Thanks! If you can add a Polars example, that would be great, but it's not required.

@deepyaman
Copy link
Contributor Author

Thanks! If you can add a Polars example, that would be great, but it's not required.

I'll just go ahead and merge it for now. A Polars example would basically just be slightly different in terms of creating the data/introducing errors (because of Polars syntax and the fact that Polars doesn't have an index), but I don't think it would add much value right now.

@deepyaman deepyaman merged commit d551921 into master Dec 24, 2024
2 checks passed
@deepyaman deepyaman deleted the deepyaman-patch-1 branch December 24, 2024 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-to-migrate Docs to migrate to new docs site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants