Skip to content

fix: Properly join URI paths in PlPath#23350

Merged
coastalwhite merged 2 commits intopola-rs:mainfrom
coastalwhite:fix/plpath-join
Jun 28, 2025
Merged

fix: Properly join URI paths in PlPath#23350
coastalwhite merged 2 commits intopola-rs:mainfrom
coastalwhite:fix/plpath-join

Conversation

@coastalwhite
Copy link
Copy Markdown
Collaborator

This also adds some tests to make sure that the behavior is somewhat consistent now.

This also adds some tests to make sure that the behavior is somewhat consistent
now.
@github-actions github-actions Bot added fix Bug fix python Related to Python Polars rust Related to Rust Polars labels Jun 28, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.77%. Comparing base (54d39f0) to head (5089ed5).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #23350      +/-   ##
==========================================
+ Coverage   80.75%   80.77%   +0.01%     
==========================================
  Files        1644     1644              
  Lines      221756   221788      +32     
  Branches     2786     2786              
==========================================
+ Hits       179088   179145      +57     
+ Misses      42006    41981      -25     
  Partials      662      662              

☔ 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 17d70e8 into pola-rs:main Jun 28, 2025
23 checks passed
@coastalwhite coastalwhite deleted the fix/plpath-join branch June 28, 2025 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fix 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