-
Notifications
You must be signed in to change notification settings - Fork 21
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
Try speeding up CI #892
base: main
Are you sure you want to change the base?
Try speeding up CI #892
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #892 +/- ##
==========================================
- Coverage 93.98% 91.27% -2.71%
==========================================
Files 134 134
Lines 9790 9790
==========================================
- Hits 9201 8936 -265
- Misses 589 854 +265
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Experiment 1: enforcing cpu threads to 1 doesn't seem to have any effect, could even be detrimental. |
Ok current state gets us to ~ 50% less, which at least gets rid of more immediate worries about 5h long runners. Merge will still be a problem though :/ |
I don't know if you are using EDIT: I just realized this hasn't been released yet. |
We are not and won't be for a while because of dependency order issues. A lot of the cost in CI here is because of coverage though, so we can squash that out for free. |
Checklist
news
entryDevelopers certificate of origin