docs(ci): persist in-container tool caches + verify conditional trigger paths#118
Conversation
…er paths Two additions to ci-runner-capacity.md: (1) persisting a tool cache written inside 'docker compose run --rm' by pointing tmpDir/cacheDirectory at a host-mounted path + actions/cache keyed on run_id with prefix restore-keys (save-fresh + restore-latest; tools self-invalidate so a stale restore is safe); (2) a green run of one conditional path (PR) does not prove the schedule/dispatch path — exercise every trigger path (workflow_dispatch only fires on the default branch, so verify after merge). Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
github-actions
Bot
added
documentation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds two new sections to the CI runner capacity documentation: one on persisting tool caches when running linters/analyzers inside docker compose run --rm, and another on verifying all trigger paths after making conditional workflow changes. The review feedback suggests correcting the Rector configuration method name to cacheDirectory(), clarifying the explanation of the restore-keys parameter in GitHub Actions, and correcting a misconception regarding how workflow_dispatch handles non-default branches.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
…view) Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
|
Two additions to
ci-runner-capacity.md(P1 — single-run measurement noise — is already covered there, so not duplicated):docker compose run --rm— caches written to the container's/tmpare discarded; pointtmpDir/cacheDirectoryat a host-mounted path +actions/cachekeyed onrun_idwith prefixrestore-keys(save-fresh + restore-latest; tools self-invalidate so a stale restore is safe; container-root writes 0644 so the runner reads on save). Measured: Lint ~145s→72s warm.schedule/workflow_dispatchpath; andworkflow_dispatchonly fires on the default branch, so that path is often only verifiable after merge.From a real CI-speed session (PHPStan/Rector caching + moving coverage to a nightly conditional).