Skip to content

Conversation

@jishnub
Copy link
Member

@jishnub jishnub commented Apr 10, 2025

These paths were not covered by tests, as flagged by code coverage.

@jishnub jishnub added the test This change adds or pertains to unit tests label Apr 10, 2025
@codecov
Copy link

codecov bot commented Apr 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.00%. Comparing base (84fd21b) to head (4a8fc62).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1280   +/-   ##
=======================================
  Coverage   92.00%   92.00%           
=======================================
  Files          34       34           
  Lines       15488    15488           
=======================================
+ Hits        14249    14250    +1     
+ Misses       1239     1238    -1     

☔ 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.

@jishnub jishnub requested a review from ViralBShah April 10, 2025 07:58
@ViralBShah ViralBShah merged commit a8fd121 into master Apr 10, 2025
4 checks passed
@ViralBShah ViralBShah deleted the jishnub/matmul_empty_tests branch April 10, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test This change adds or pertains to unit tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants