Repository navigation
Conversation
Adds the AI review and review gate callers, a PR template with Evidence and What to try, and the feature review Slack ping in the nightly build. CODEOWNERS is removed: with no mandatory approval it only auto-requested the whole team on every PR; the needs-human reviewer comes from the PROJECT_REVIEWER repository variable instead. Shared workflow refs move to 2.8.0, the release that ships these workflows. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
ai-review.yml and review-gate.yml become code-review.yml: the AI review runs on every push of a ready PR, the gate on pull_request_target and review events. Nightly and on-demand builds keep their original permissions; the Slack ping is a separate feature-review job. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
A skipped job leaves a check run named after it, and a required check named review-gate would accept that skipped run. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The shared workflow picks the events for the review and the gate and takes the reviewer from PROJECT_REVIEWER; the build workflows send the feature review message themselves. Refs are on main until the next shared workflows release. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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.
Summary
The template adopts the new iOS code review process: an AI review bot on every push to a PR into
develop, human approval only for red zones andneeds-human, evidence in every PR, and feature review in the app after nightly and on-demand builds.Changes
.github/workflows/code-review.yml: one job calling the shareduniversal-cloud-ai-pr-reviewwithsecrets: inherit. The shared workflow runs the review and thereview-gatestatus on the right events and takes the reviewer fromPROJECT_REVIEWER.nightly-build.yml,on-demand-build.yml: passSLACK_WEBHOOK_FEATURE_REVIEW. The shared build workflows post PRs labeledfeature-completeto Slack and mention the reviewer through theSLACK_USER_IDSorg variable..github/pull_request_template.md: Summary, Evidence, What to try.AGENTS.md: PR rule,## Red zonessection (read from the base branch),/ios-reviewrow.CODEOWNERSremoved: without a mandatory approval it only auto-requested the whole iOS team on every PR.mainuntil the next shared release (feat(review): run the whole code review from one caller job .github#139).Evidence
actionlintis clean. The samecode-review.ymlruns on a consumer project.Notes
Merge after the next
.githubrelease, with these refs moved frommainto that tag. Per project: the repository variablePROJECT_REVIEWERand the ruleset command from the handbook (Engineering-Handbook#121).🤖 Generated with Claude Code