-
Notifications
You must be signed in to change notification settings - Fork 3
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
Testing #16
Testing #16
Conversation
Codecov Report
@@ Coverage Diff @@
## main #16 +/- ##
==========================================
+ Coverage 83.68% 87.17% +3.49%
==========================================
Files 1 1
Lines 190 195 +5
==========================================
+ Hits 159 170 +11
+ Misses 31 25 -6
Continue to review full report at Codecov.
|
This is good work, but not quite ready to merge into main. We need to code review and ensure we're not accidentally loosing any of main's testing. Specifically, we need to be reasonably sure that bugs that we explicitly test for in main will also be effectively caught here (i.e. guaranteed or very high probability). The API improvements discovered and implemented here have just been added to main in c06207a and 0ed9578 for the 0.3.0 release. |
… a strict improvement.
559955b
to
00218eb
Compare
That was a rebase |
…ber major.minor (and no overwrite). No longer automatically attempts teh failed migration from 1.8.0-DEV to 1.0.1
8fc306c
to
0a91430
Compare
Looks good to me! |
- remove known issues section because the package migration part is fixed & the windows disclaimer is long, not particularly insightful or helpful, and not an actual issue as far as I'm aware of. - update to reflect docstring change from testing PR #16 - add example usage to install release candidates and nightly
No description provided.