search

morelinq / MoreLINQ

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

Refactor `RandomSubset` internals for DRY, removing closure #1049

Closed viceroypenguin closed 7 months ago

viceroypenguin commented 7 months ago

This PR updates the implementation of RandomSubsetImpl to remove the unnecessary closure and DRY the instantiation code.

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a0f72ab) 92.50% compared to head (93ca56b) 92.50%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1049 +/- ## ========================================== - Coverage 92.50% 92.50% -0.01% ========================================== Files 112 112 Lines 3404 3401 -3 Branches 1056 1056 ========================================== - Hits 3149 3146 -3 Misses 189 189 Partials 66 66 ```

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