8000 test: Update tests to be index type agnostic by JakubValtar · Pull Request #24891 · pola-rs/polars · GitHub
[go: up one dir, main page]
Skip to content

test: Update tests to be index type agnostic #24891

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.

Sign up for GitHub

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

Jump to bottom
Merged
coastalwhite merged 1 commit into from
Oct 16, 2025
Merged

test: Update tests to be index type agnostic #24891

coastalwhite merged 1 commit into from
Oct 16, 2025

Conversation

@JakubValtar
Copy link
Collaborator
@JakubValtar JakubValtar commented Oct 15, 2025

This PR:

@github-actions github-actions bot added internal An internal refactor or improvement python Related to Python Polars rust Related to Rust Polars labels Oct 15, 2025
@codecov
Copy link
codecov bot commented Oct 15, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.75%. Comparing base (afdbf30) to head (71d9bbf).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #24891      +/-   ##
==========================================
+ Coverage   81.72%   81.75%   +0.03%     
==========================================
  Files        1701     1701              
  Lines      233901   233901              
  Branches     2996     2996              
==========================================
+ Hits       191157   191230      +73     
+ Misses      41982    41909      -73     
  Partials      762      762

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coastalwhite coastalwhite merged commit 1090f1f into pola-rs:main Oct 16, 2025
16 checks passed
@JakubValtar JakubValtar deleted the update-tests-index-type branch October 16, 2025 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nameexhaustion nameexhaustion nameexhaustion approved these changes

@ritchie46 ritchie46 Awaiting requested review from ritchie46 ritchie46 is a code owner

@c-peters c-peters Awaiting requested review from c-peters c-peters is a code owner

@alexander-beedie alexander-beedie Awaiting requested review from alexander-beedie alexander-beedie is a code owner

@MarcoGorelli MarcoGorelli Awaiting requested review from MarcoGorelli MarcoGorelli is a code owner

@reswqa reswqa Awaiting requested review from reswqa reswqa is a code owner

Assignees

No one assigned

Labels

internal An internal refactor or improvement python Rel 4C6A ated to Python Polars rust Related to Rust Polars

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JakubValtar @nameexhaustion @coastalwhite
0