-
Notifications
You must be signed in to change notification settings - Fork 64
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
Test infra improvements #461
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #461 +/- ##
======================================
Coverage 94.2% 94.2%
======================================
Files 23 23
Lines 1728 1728
======================================
Hits 1629 1629
Misses 99 99 |
2e9fbe3
to
e33b583
Compare
@jayqi Ready, but not urgent. Thinking this can be just post-release since it is not really user-facing. I hope it improves test reliability, but in my experience failures seem to be related in some way to resource load on shared GH runners (failures seem to cluster in time). It just needs a history change added, but didn't want to collide with #463 |
This reverts commit 197a058.
4d293f1
to
8045ed0
Compare
I'll say this is "ready" once I get 5 error-free passes.
A few improvements: