Wire up React Native CI workflows#55
Merged
Merged
Conversation
🔍 Denylist Analysis ResultsAnalyzed 2 packages. .github/workflows/rn-test.yml✅ Known Packages - Approved for Use (2)
For any questions or to provide feedback, please reach out to #help-bumperbot. Generated by Bumperbot. |
Juanita-Dash
force-pushed
the
rn-ci-wiring
branch
3 times, most recently
from
b6d630a to
e3ea5d0
Compare
Juanita-Dash
requested review from
a team,
kiftio and
markmur
and removed request for
kiftio
Juanita-Dash
force-pushed
the
rn-ci-wiring
branch
from
e3ea5d0 to
05db988
Compare
React Native — Coverage Report
|
Juanita-Dash
force-pushed
the
rn-ci-wiring
branch
2 times, most recently
from
fbd9a03 to
1429ee9
Compare
Adds six rn-*.yml reusable workflows under .github/workflows/, mirroring the source repo's CI surface and following the existing android-*/swift-* monorepo conventions (em-dash names, pinned action SHAs, workflow_call + workflow_dispatch triggers, working-directory: platforms/react-native, shop/setup-javascript-action for the JS toolchain). Jobs (ordered "tests first, lint last" to match the swift section): - rn-test jest (with coverage comment on PRs) - rn-test-android pnpm sample test:android (JDK 22 / Gradle) - rn-test-ios pnpm sample test:ios (vars.MACOS_RUNNER + cocoapods) - rn-license-headers ./scripts/copy_license --check - rn-check-packed-files module build + compare-snapshot - rn-lint SwiftLint + module build + module/sample eslint ci.yml: wires each new workflow into the orchestrator, extends the reactNative paths filter to watch the new files, and adds them to ci-required.needs so they participate in the aggregated status check. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Juanita-Dash
force-pushed
the
rn-ci-wiring
branch
from
1429ee9 to
a1195b1
Compare
markmur
approved these changes
May 11, 2026
Contributor
markmur
left a comment
markmur
left a comment
There was a problem hiding this comment.
Looks good to go once CI is passing
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
3 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.
Fixes https://github.com/shop/issues-checkout-kit/issues/907
Wires up React Native CI in the monorepo so PRs touching
platforms/react-native/**get the same automated signal thatplatforms/android/**andplatforms/swift/**already do. Mirrors the source repo'sci.ymland follows the existingandroid-*/swift-*patterns in this repo.What's added
Six reusable workflows (
rn-lint,rn-test,rn-test-android,rn-test-ios,rn-license-headers,rn-check-packed-files), wired intoci.yml's paths filter andci-requiredaggregator. Plus fourdependabot.ymlentries (npm × 3 + bundler) for the RN tree.Note: one-line
.gitignoreexception for iOS entitlementsAdded
!platforms/react-native/sample/ios/ReactNative/ReactNative.entitlementsto the root.gitignore. The file was silently dropped during PR #31's verbatim import because the monorepo's**/*.entitlementsrule excluded it. It's source code (associated domains + Apple Pay merchant ID), not a secret — Xcode requires it at build time andrn-test-iosfails without it.Scope: build + test only
Publish (
rn-deploy.yml) is a follow-up PR. Blocked on: monorepo tag-naming convention for RN releases, authorizing this repo to publish the npm package (currently only the source repo is), and coordinating the cutover from the source repo's publish workflow. Final package name also depends on #27 landing.🤖 Generated with Claude Code