morelinq / MoreLINQ

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

Cleanup `Partition` Nullability #881

Closed viceroypenguin closed 1 year ago

viceroypenguin commented 1 year ago

This PR adds to https://github.com/morelinq/MoreLINQ/issues/803. It uses nullable TKey values in the implementation to more accurately express the code.

codecov[bot] commented 1 year ago

Codecov Report

Merging #881 (2c8b409) into master (c3a7094) will increase coverage by 0.20%. The diff coverage is 100.00%.

:exclamation: Current head 2c8b409 differs from pull request most recent head 26b9656. Consider uploading reports for the commit 26b9656 to get more accurate results

@@            Coverage Diff             @@
##           master     #881      +/-   ##
==========================================
+ Coverage   92.38%   92.58%   +0.20%     
==========================================
  Files         110      108       -2     
  Lines        3441     3440       -1     
  Branches     1020     1024       +4     
==========================================
+ Hits         3179     3185       +6     
+ Misses        200      194       -6     
+ Partials       62       61       -1     
Impacted Files Coverage Δ
MoreLinq/Partition.cs 100.00% <100.00%> (+1.75%) :arrow_up:
MoreLinq/Pad.cs 100.00% <0.00%> (ø)
MoreLinq/FallbackIfEmpty.cs 88.67% <0.00%> (ø)
MoreLinq/Assume.cs
MoreLinq/Debug.cs
MoreLinq/Lookup.cs 64.12% <0.00%> (+2.25%) :arrow_up:

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