ci: cancel superseded runs, cache bun downloads and transform output, bound matrix cells#72
Open
danfry1 wants to merge 1 commit into
Open
ci: cancel superseded runs, cache bun downloads and transform output, bound matrix cells#72danfry1 wants to merge 1 commit into
danfry1 wants to merge 1 commit into
Conversation
… bound matrix cells - Add concurrency groups to ci.yml and native-rn-matrix.yml: a force-push to a PR branch previously left the superseded run (5 gate legs + 12 matrix cells) executing to completion alongside the new one. Main pushes stay uncancelled so every landed commit gets a full verdict. - Cache bun's package-download store keyed on the lockfile; installs still verify against --frozen-lockfile, they just skip re-downloading tarballs. - Persist vitest-native's own on-disk caches (Babel transform of the RN graph + V8 compile cache) across runs, keyed per OS/Node leg and per RN × Vitest matrix cell. The entries are content/version-keyed internally, so a stale restore is a miss, never a wrong hit. (Effective once the cache moves under node_modules/.cache — see the transform-cache change; the step is a no-op before that.) - Bound matrix cells with timeout-minutes: 30 — a hung cell previously burned the 360-minute default; healthy cells finish well under 15.
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
1 participant
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.
Problem
Four CI-hygiene gaps:
Change
concurrencygroups onci.ymlandnative-rn-matrix.ymlwithcancel-in-progresson non-main refs; pushes to main stay uncancelled so every landed commit gets a full verdict.actions/cache(SHA-pinned, v4.3.0) for bun's package-download store, keyed on the lockfile. Installs still verify with--frozen-lockfile; only the tarball downloads are skipped.actions/cacheforpackages/vitest-native/node_modules/.cache/vitest-native(the Babel transform + V8 compile caches), keyed per OS/Node leg in the full gate and per RN × Vitest cell in the matrix. The cache's entries are content- and version-keyed internally, so a stale restore can only miss, never serve a wrong hit. This step becomes effective once the transform-cache relocation (perf/transform-cache) lands — before that the path doesn't exist and the step is a harmless no-op, so the two PRs can land in either order.timeout-minutes: 30on the matrix job (healthy cells finish well under 15).All new expressions use trusted contexts only (
github.ref,runner.os,matrix.*,hashFiles); no event-payload data reachesrun:commands.