Repository navigation
ci(actions): add a path-aware React Native PR gate - #299
Conversation
Classify pull request changes before selecting package, Android, and iOS jobs while keeping the workflow and existing job names stable. Fail closed for protected branches, workflow and dependency files, ambiguous paths, renames, copies, and empty diffs, then aggregate every selected result through an always-running PR Gate. Keep Google Chat transport outside validation, exempt safe documentation from code-owner matching, and group patch/minor dependency updates by ecosystem. Co-Authored-By: Codex <noreply@openai.com>
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.
Tip: disable this comment in your organization's Code Review settings.
🛣️ Rokt Platform Paved Roads Scan Complete✅ No paved-road issues detected Scanned files against 188 rules. · Reviewed commit: 📜 Scan history (2 scans)
ℹ️ About Rokt Platform Paved Roads AgentThis scan was performed by the Rokt Platform Paved Roads Agent, an automated paved-road standards review tool for pull requests. See the User Guide for details on severity levels, feedback reactions, and scan configuration. Reply |
🔒 Security Scan Complete✅ No security issues detected Scanned files against 88 rules. · Reviewed commit: 📜 Scan history (2 scans)
ℹ️ About PR Security AgentThis scan was performed by PR Security Agent, an automated security review tool for pull requests. See the User Guide for details on severity levels, feedback reactions, and scan configuration. Reply |
nickolas-dimitrakas
left a comment
There was a problem hiding this comment.
Reviewed — CI green, no open questions. Approving so this can merge directly.
Background
Rokt.Widgetand builds both Android and iOS for every change. In the audit sample, 8 of 21 merged PRs were CI-only and another 4 of 21 were Android-only, so more than half could avoid either both platform builds or the unrelated iOS build.What Has Changed
Rokt.Widget/android/**: package plus Android;Rokt.Widget/ios/**: package plus iOS;if: always()PR Gatethat verifies the classifier, Trunk, package, Android, and iOS results against the selected plan. Skipped jobs are accepted only when the classifier explicitly did not select them.continue-on-error, after the PR gate has already passed.Screenshots/Video
Checklist
Additional Notes
bash .github/scripts/select-pr-jobs.spec.sh— 23 checks, 0 failures.full, so this PR cannot reduce its own validation.go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.7 .github/workflows/pull-request.ymlpassed.bash -npassed for both classifier scripts; Ruby parsed the workflow and Dependabot YAML;git diff --checkpassed.npm ci,npm run lint,npm run build, andnpm run build:pluginpassed inRokt.Widget. The install still reports the repository's existing dependency audit findings; this PR does not change the npm graph.PR Gate, enable auto-merge, and enable update-branch support. Existing job names remain available during the transition.Co-Authored-By: Codex noreply@openai.com