ci: run the native smoke suite on every pull request on macOS and Linux - #902
Closed
janicduplessis wants to merge 2 commits into
Closed
janicduplessis wants to merge 2 commits into
janicduplessis wants to merge 2 commits into
Conversation
…ux, warm from the cross-run cache e2e-native.yml gains a smoke suite: run-native-e2e.mjs --smoke, Expo only, one iOS job on xcode-27 and one Android job on ubuntu with KVM, selected for any pull request that does not carry the e2e-native label. Labeled pull requests, the nightly and dispatch keep their suites. Documentation-only pull requests skip the workflow. The smoke restores the same cross-run build cache the loop does, so a change that leaves the native fingerprint alone installs from cache; the Windows lane in ci.yml gets the same cache.
Collaborator
Author
|
Superseded: the gate is changing to push-to-main plus opt-in labels (e2e-smoke, e2e-loop, e2e-caches, e2e-pool, e2e-all) with the Windows job moving into e2e-native.yml; a fresh PR against #901 follows. |
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.
Closes #901. Stacked on #899 (base branch set to it; will retarget main once #899 merges).
Pull requests never built or launched anything natively on macOS or Linux:
e2e-native.ymlran nightly, on dispatch, or on a pull request carrying thee2e-nativelabel, which none has carried. This adds a smoke suite to that workflow, driven byrun-native-e2e.mjs --smokefrom #899: one worktree,stim start, build, install, launch, readiness,stim stop, cleanup checks. Expo framework only (bare is already excluded on iOS 27), so oneios expo (smoke)job onxcode-27and oneandroid expo (smoke)job on Ubuntu with KVM, blocking.Gate. The job-level
if:gates are replaced by the matrixsuiteexpression: an unlabeled pull request selectssmoke, a labeled oneloop, the nightlyloop, and dispatch its input (smokeis a new option). Pull requests touching onlydocs/,website/or Markdown skip the workflow viapaths-ignore.Speed. The smoke restores the same cross-run build cache the loop already persists with
actions/cache, so a pull request that leaves the native fingerprint alone installs from cache (about 8 minutes iOS, 5 Linux); one that changes it pays the cold build. The Windows Android lane inci.ymlgets the same cache (D:\e\build-cache, keyed like the others) and the same warm-cache detection, bringing its typical run from ~25 to ~14 minutes, the emulator boot being the floor.docs/e2e-and-ci.mdand both workflow headers describe the new gate.