-
Notifications
You must be signed in to change notification settings - Fork 295
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
Indicate additional breaking changes + docstring fixes #573
Conversation
Codecov Report
@@ Coverage Diff @@
## master #573 +/- ##
==========================================
+ Coverage 94.26% 94.55% +0.28%
==========================================
Files 29 29
Lines 6348 6884 +536
Branches 675 874 +199
==========================================
+ Hits 5984 6509 +525
- Misses 226 236 +10
- Partials 138 139 +1
Continue to review full report at Codecov.
|
Does the wording look good to you @jmwright ? |
@adam-urbanczyk Looks good to me. Is there a specific PR that each breaking change can be tagged with, or is it too hard to pin the changes down to just one PR? |
Good point, added those. |
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.
Looks good, thanks @adam-urbanczyk
OK, merging. |
Will resolve #572