Repository navigation
ci: skip unrelated label events without cancelling real runs; ZZDesign tests in the ios job - #82
Merged
Merged
Conversation
…m; ZZDesign tests in the ios job Adding a label other than ios-ci (apps-ci) or run-e2e/ios-ci (apps-e2e) skips every job and uses its own concurrency group, so it no longer cancels or duplicates the pull request's real run (26 of 82 Apps CI runs were cancelled duplicates). The ios job also runs the ZZDesign package tests. ci.yml's header comment no longer says apps-ci is to be added. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
scrimshawlife-ctrl
approved these changes
Oct 17, 2026
scrimshawlife-ctrl
left a comment
scrimshawlife-ctrl
left a comment
Contributor
There was a problem hiding this comment.
Approved by Danny 2026-10-16 23:57 PT
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
2 participants
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.
From the 2026-10-16 zzThat audit.
cancel-in-progressthen cancelled the real run: 26 of the last 82 runs were cancelled duplicates (for example Point T096 at the merged unknown-enum half #80). Now alabeledevent for a label other thanios-ciskips every job and runs in its own concurrency group, so it can't cancel anything. Apps E2E does the same for labels other thanrun-e2eorios-ci.iosjob now runs the ZZDesign package tests, which CI never ran before.ci.yml: the header comment no longer saysapps-ci.yml"is added by the build".Not changed (your call: ZQ27 spend): the
iosjob still runs only with theios-cilabel or by hand. So #63, #64, #65, #76 and #79 merged iOS code with iOS CI skipped, andmainis never re-tested after a merge. Options:iosonpushtomain, about one macOS run per merge;ios-cion PRs that touchapps/ios/**;Recommend (a).
The YAML parses and
lint-specs.pypasses. Actions behavior can only be confirmed on GitHub.🤖 Generated with Claude Code