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
ci: run all linter benchmarks #3222
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @overlookmotel and the rest of your teammates on Graphite |
CodSpeed Performance ReportMerging #3222 will not alter performanceComparing Summary
Benchmarks breakdown
|
Github action queues the jobs, adding too many linter benchmarks will drag out these jobs. Let me put this on hold. Let me fix the generic monomorphization problem first, I'm getting there. |
bce02fe
to
31a02a5
Compare
OK, no problem. Actually we should probably drop the 5th benchmark ( My logic was that now that the jobs don't include building the benchmark and run in ~1 minute, they don't gum up the queue for long, so contention is less of a concern. |
Feels like 2 linter benchmarks is enough for me, closing. |
After #3221 we can re-enable the linter benchmarks which were disabled in #3172, as they won't hog so much CI time any more.