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

[CHORE] Tidy typing for remaining binary ops: logical, comp #1124

Merged
merged 9 commits into from
Jul 6, 2023

Conversation

xcharleslin
Copy link
Contributor

@xcharleslin xcharleslin commented Jul 5, 2023

Similar to #1114, for the remaining binary ops (logical and comp).

Also cleans up the macros in SeriesLike binary_ops as well.

@codecov
Copy link

codecov bot commented Jul 6, 2023

Codecov Report

Merging #1124 (7fe39be) into main (a4dbd62) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 7fe39be differs from pull request most recent head f16f085. Consider uploading reports for the commit f16f085 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1124      +/-   ##
==========================================
- Coverage   88.51%   88.49%   -0.02%     
==========================================
  Files          54       54              
  Lines        5476     5476              
==========================================
- Hits         4847     4846       -1     
- Misses        629      630       +1     

see 1 file with indirect coverage changes

@xcharleslin xcharleslin changed the title Charles/typing2 [CHORE] Tidy typing for remaining binary ops: logical, comp Jul 6, 2023
@github-actions github-actions bot added the chore label Jul 6, 2023
@xcharleslin xcharleslin marked this pull request as ready for review July 6, 2023 20:54
@xcharleslin xcharleslin enabled auto-merge (squash) July 6, 2023 20:54
@xcharleslin xcharleslin merged commit 02d31f8 into main Jul 6, 2023
19 of 20 checks passed
@xcharleslin xcharleslin deleted the charles/typing2 branch July 6, 2023 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant