Skip to content

NewMatrix(...,ncols,list) changed to NewMatrix(...,list,ncols) - #6360

Closed
cdwensley wants to merge 2 commits into
masterfrom
swap
Closed

NewMatrix(...,ncols,list) changed to NewMatrix(...,list,ncols)#6360
cdwensley wants to merge 2 commits into
masterfrom
swap

NewMatrix(...,ncols,list) changed to NewMatrix(...,list,ncols)

9cac120
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded May 1, 2026 in 1s

75.94% (-2.69%) compared to b04e9ba

View this Pull Request on Codecov

75.94% (-2.69%) compared to b04e9ba

Details

Codecov Report

❌ Patch coverage is 39.13043% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.94%. Comparing base (b04e9ba) to head (9cac120).

Files with missing lines Patch % Lines
lib/matobj.gi 42.85% 8 Missing ⚠️
lib/matobjnz.gi 0.00% 3 Missing ⚠️
lib/matobjplist.gi 40.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6360      +/-   ##
==========================================
- Coverage   78.63%   75.94%   -2.69%     
==========================================
  Files         684      644      -40     
  Lines      292689   277950   -14739     
  Branches     8686     7883     -803     
==========================================
- Hits       230155   211103   -19052     
- Misses      60726    65309    +4583     
+ Partials     1808     1538     -270     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.