Skip to content

Commit 907e8fc

Browse files
ci: fix false-failure risk — use xcodebuild exit code, not error: grep
1 parent a1d06ac commit 907e8fc

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/iphone-duo-ios-check.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,17 +70,17 @@ jobs:
7070
- name: Build RNTester Debug for iOS Simulator (compiles RCTDeviceInfo.mm)
7171
working-directory: ${{ env.APP_IOS_DIR }}
7272
run: |
73+
# Use xcodebuild's own exit code, not log-text grepping — a "error:"
74+
# substring can show up harmlessly in retry/warning noise even on a
75+
# build that ultimately succeeds.
76+
set -o pipefail
7377
xcodebuild \
7478
-workspace "$XCODE_PROJECT" \
7579
-scheme "$XCODE_SCHEME" \
7680
-configuration Debug \
7781
-sdk iphonesimulator \
7882
-destination 'generic/platform=iOS Simulator' \
7983
build 2>&1 | tee /tmp/xcodebuild.log
80-
if grep -qE "^\*\* BUILD FAILED \*\*|error:" /tmp/xcodebuild.log; then
81-
echo "::error::xcodebuild failed — see log above, especially any errors in RCTDeviceInfo.mm."
82-
exit 1
83-
fi
8484
echo "RNTester (Debug, iphonesimulator) built cleanly, including the modified RCTDeviceInfo.mm."
8585
8686
- name: Upload build log

0 commit comments

Comments
 (0)