-
Notifications
You must be signed in to change notification settings - Fork 61
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
API change stop_strategy to stopping_strategy #274
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!! :) just a few nitpicks.
Co-authored-by: Thomas Moreau <thomas.moreau.2010@gmail.com>
Codecov Report
@@ Coverage Diff @@
## main #274 +/- ##
==========================================
- Coverage 62.66% 60.80% -1.86%
==========================================
Files 41 41
Lines 2100 2202 +102
Branches 372 401 +29
==========================================
+ Hits 1316 1339 +23
- Misses 688 766 +78
- Partials 96 97 +1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few more nitpicks :)
But then we are good to go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! thx @bmalezieux
Issue #273