[Phase 4] Build React Native iOS E2E IPA artifact#361
Merged
kieran-osgood-shopify merged 2 commits intoJul 9, 2026
Merged
Conversation
This was referenced Jun 29, 2026
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Jun 29, 2026
2093783 to
8d428c0
Compare
47a512b to
61cf06e
Compare
455ab75 to
b1b2dfd
Compare
883a195 to
3d0b744
Compare
68ea79c to
12fb9f6
Compare
b1b75a8 to
31ed453
Compare
3dcb803 to
e2d2b87
Compare
31ed453 to
3cd4001
Compare
e2d2b87 to
e5a9bbe
Compare
97d9280 to
ffb6424
Compare
e5a9bbe to
17f737e
Compare
3d95d77 to
5e92742
Compare
6d6a642 to
9feda34
Compare
3ef18ef to
7c78f0e
Compare
9feda34 to
a842383
Compare
7c78f0e to
293b136
Compare
e534ccd to
daf0413
Compare
293b136 to
17886cc
Compare
daf0413 to
d32656c
Compare
47daa4f to
ccbf8f3
Compare
05ea674 to
349dc1b
Compare
ccbf8f3 to
345c62f
Compare
349dc1b to
cd2d095
Compare
345c62f to
a744417
Compare
kiftio
reviewed
Jul 8, 2026
| title: Create React Native iOS artifact placeholder | ||
| title: Build React Native iOS IPA artifact | ||
| timeout: 5400 | ||
| no_output_timeout: 5400 |
Contributor
There was a problem hiding this comment.
should we make these different?
We could keep timeout high, but drop no_output to 1800 or something to detect a hang?
| - restore-cache@3: | ||
| inputs: | ||
| - key: &rn_ruby_gems_cache_key |- | ||
| rn-ruby-gems-{{ .OS }}-{{ .Arch }}-ruby-3.4.8-{{ checksum "platforms/react-native/sample/Gemfile.lock" }} |
Contributor
There was a problem hiding this comment.
Is there a {{ .RubyVersion }} or worth extracting a variable...
Actually is it needed at all? The Gemfile.lock includes Ruby version, so a change would invalidate the cache when the ruby version changes without it
cd2d095 to
ba7892e
Compare
a744417 to
32ba8de
Compare
11 tasks
32ba8de to
b1d0394
Compare
3b48186 to
f6a559a
Compare
markmur
approved these changes
Jul 8, 2026
Assisted-By: devx/6c1e3ad5-96c8-4972-b087-da7ff7b195c3
Assisted-By: devx/9ae89f69-61a5-46e3-8faf-006d105b0cdd
This was referenced Jul 10, 2026
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.

Stack Context
This is Phase 4 of the BrowserStack Maestro E2E stack for Checkout Kit, stacked on the RN Android artifact workflow.
Parent issue: https://github.com/shop/issues-checkout-kit/issues/1098
Epic: https://github.com/shop/issues-checkout-kit/issues/1084
What?
Replaces the React Native iOS placeholder artifact workflow with a signed device archive/export path:
scripts/setup_storefront_envxcodebuild archiveforiphoneos.ipaviaxcodebuild -exportArchiveE2E_REACT_NATIVE_IOS_APP_PATHWhy?
BrowserStack App Automate requires an
.ipafor iOS real-device Maestro execution. RN E2E should validate the released native CocoaPods artifacts used by the React Native sample/module configuration, not local in-repo Swift SDK sources.Validation