-
Notifications
You must be signed in to change notification settings - Fork 411
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
min max wrapper #556
min max wrapper #556
Conversation
for more information, see https://pre-commit.ci
@SkafteNicki seems to be a duplicate to #506 |
Co-authored-by: Nicki Skafte Detlefsen <[email protected]>
@janhenriklambrechts how is it going here, ready for review? |
@Borda not yet finished, but will work on it this afternoon -- so expect a clean PR by tomorrow :) |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Codecov Report
@@ Coverage Diff @@
## master #556 +/- ##
=====================================
- Coverage 95% 95% -0%
=====================================
Files 152 153 +1
Lines 5362 5396 +34
=====================================
+ Hits 5102 5134 +32
- Misses 260 262 +2 |
@Borda @SkafteNicki, I have identified the difference between For,
The corresponding
These results make sense to me.
These values correspond to the ones above, but omit half and thus lead to wrong |
@SkafteNicki mind have look ^^ 🐰 |
for more information, see https://pre-commit.ci
Before submitting
What does this PR do?
Fixes #49
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃