Skip to content

Cast binary op result to output type#976

Merged
ehsantn merged 1 commit intomainfrom
ehsan/binop_res_type
Dec 15, 2025
Merged

Cast binary op result to output type#976
ehsantn merged 1 commit intomainfrom
ehsan/binop_res_type

Conversation

@ehsantn
Copy link
Collaborator

@ehsantn ehsantn commented Dec 15, 2025

Changes included in this PR

Avoids data type mismatch runtime errors (e.g. for df.A + 10 where the column type is int32). Fixes Narwhals test issues.

Testing strategy

Narwhals tests.

User facing changes

Bug fix

Checklist

  • Pipelines passed before requesting review. To run CI you must include [run CI] in your commit message.
  • I am familiar with the Contributing Guide
  • I have installed + ran pre-commit hooks.

@codecov
Copy link

codecov bot commented Dec 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.71%. Comparing base (c33fbb5) to head (10f0c8e).
⚠️ Report is 164 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #976      +/-   ##
==========================================
+ Coverage   66.68%   67.71%   +1.03%     
==========================================
  Files         186      195       +9     
  Lines       66795    67686     +891     
  Branches     9507     9629     +122     
==========================================
+ Hits        44543    45836    +1293     
+ Misses      19572    19027     -545     
- Partials     2680     2823     +143     

Copy link
Contributor

@IsaacWarren IsaacWarren left a comment

Choose a reason for hiding this comment

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

Nice!

@ehsantn ehsantn merged commit bc1bf21 into main Dec 15, 2025
25 of 30 checks passed
@ehsantn ehsantn deleted the ehsan/binop_res_type branch December 15, 2025 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants