search

mljs / matrix

Matrix manipulation and computation library
https://mljs.github.io/matrix/
MIT License
353 stars 54 forks source link

Fix optional argument for `norm()` #165

Closed rotu closed 11 months ago

rotu commented 1 year ago

Previously, TypeScript did not like an argumentless norm()

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 85.71% and project coverage change: -0.01% :warning:

Comparison is base (bb02eb3) 71.28% compared to head (5363a1a) 71.27%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #165 +/- ## ========================================== - Coverage 71.28% 71.27% -0.01% ========================================== Files 34 34 Lines 5300 5295 -5 Branches 850 849 -1 ========================================== - Hits 3778 3774 -4 + Misses 1517 1516 -1 Partials 5 5 ``` | [Files Changed](https://app.codecov.io/gh/mljs/matrix/pull/165?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mljs) | Coverage Δ | | |---|---|---| | [src/matrix.js](https://app.codecov.io/gh/mljs/matrix/pull/165?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mljs#diff-c3JjL21hdHJpeC5qcw==) | `79.55% <85.71%> (-0.01%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.