-
Notifications
You must be signed in to change notification settings - Fork 170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
172: Max and Min #263
172: Max and Min #263
Conversation
@svendiedrichsen Thanks for your report. I think fixed them now. Could you check it? |
Codecov Report
@@ Coverage Diff @@
## master #263 +/- ##
============================================
+ Coverage 73.5% 73.98% +0.48%
- Complexity 387 398 +11
============================================
Files 124 126 +2
Lines 1200 1230 +30
Branches 41 47 +6
============================================
+ Hits 882 910 +28
- Misses 297 298 +1
- Partials 21 22 +1
Continue to review full report at Codecov.
|
@rultor merge |
Tests are looking good. I was just thinking if the original NoSuchElementException isn't already good enough. @fabriciofx: WDYT? |
@svendiedrichsen Makes sense. I'll do it. Thanks! |
As per #172:
Max
introduced.Min
introduced.