Repository navigation
Static checks: run the checks as background steps - #3739
Closed
reaperhulk wants to merge 1 commit into
Closed
reaperhulk wants to merge 1 commit into
reaperhulk wants to merge 1 commit into
Conversation
reaperhulk
force-pushed
the
claude/faster-static-checks
branch
from
October 30, 2026 05:52
24a8418 to
2d94121
Compare
The static checks job ran its 15 checks one after another, 20.6 s of its 42 s. Their CPU time is about 33 s over the runner's four cores, so each is now a background step, slowest first (at most 10 run at once, the rest queue), and a wait-all at the end fails if any of them failed. Each keeps its own step and log. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015ajQWwSSE6PJMaZ9NzWaSV
reaperhulk
force-pushed
the
claude/faster-static-checks
branch
from
October 30, 2026 05:59
2d94121 to
b128b68
Compare
reaperhulk
marked this pull request as draft
October 30, 2026 06:02
reaperhulk
marked this pull request as ready for review
October 30, 2026 12:40
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Static checks job took 42 s on the last merge-queue run (41004903518): 18 s checking out the repository, then 20.6 s running its 15 checks one after another. Their CPU time is about 33 s, which spread over the runner's four cores is about 8 s.
Change
background: truestep, and await-all:step at the end fails the job if any of them failed. Each check keeps its own step and log.run_unit_tests.pycheck_lean_speed.pycheck_variants.pycheck_lean_imports.pycheck_mcdt.pyThe checkout stays as it was. A sparse checkout that left out
vectors/made it slower: 25 s instead of 18 s, because it fetches each of this job's 27,000 files on demand.Result
On this PR's run (41011468451), the job took 34 s instead of 42 s:
A step's own time now includes waiting for a core while the others run. For example,
check_arch_gates.pyshows 9 s instead of 0 s, so judge the change by the job's total time, not by single steps.Testing
all-greenpassed in this PR's run.ci.ymlchanges.🤖 Generated with Claude Code
https://claude.ai/code/session_015ajQWwSSE6PJMaZ9NzWaSV