search

morelinq / MoreLINQ

Extensions to LINQ to Objects
https://morelinq.github.io/
Apache License 2.0
3.7k stars 415 forks source link

Fix `Memoize` Implementation #898

Closed viceroypenguin closed 1 year ago

viceroypenguin commented 2 years ago

This PR does three things: 1) Correct issues in Memoize implementation; 2) Update tests for Memoize and Cartesian to reflect updated expectations; 3) Update TestingSequence with better expectations for how a testing sequence should be used.

Fixes #889

codecov[bot] commented 2 years ago

Codecov Report

Merging #898 (82e213b) into master (c3a7094) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #898   +/-   ##
=======================================
  Coverage   92.38%   92.38%           
=======================================
  Files         110      110           
  Lines        3441     3441           
  Branches     1020     1020           
=======================================
  Hits         3179     3179           
  Misses        200      200           
  Partials       62       62

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

viceroypenguin commented 1 year ago

Upon further review, this is the incorrect path to address concerns anyway.