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

fix(python): Raise if single argument form in replace/replace_strict is not a mapping #18492

Merged
merged 2 commits into from
Sep 9, 2024

Conversation

henryharbeck
Copy link
Contributor

Closes #18185

Copy link

codecov bot commented Aug 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.86%. Comparing base (fb38919) to head (ece7405).
Report is 96 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #18492      +/-   ##
==========================================
- Coverage   79.91%   79.86%   -0.05%     
==========================================
  Files        1495     1501       +6     
  Lines      200261   202038    +1777     
  Branches     2867     2870       +3     
==========================================
+ Hits       160033   161359    +1326     
- Misses      39682    40132     +450     
- Partials      546      547       +1     

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

@henryharbeck
Copy link
Contributor Author

Hi @ritchie46, @alexander-beedie, @MarcoGorelli
May I please request a review on this PR.
Thanks

Copy link
Member

@ritchie46 ritchie46 left a comment

Choose a reason for hiding this comment

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

Thanks!

@ritchie46 ritchie46 merged commit 433d6c0 into pola-rs:main Sep 9, 2024
15 checks passed
@henryharbeck henryharbeck deleted the replace_no_default branch September 9, 2024 11:29
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expr.replace and Expr.replace_strict set "NO_DEFAULT" as a value
2 participants