Skip to content

Commit 81a5f76

Browse files
ci: fail fast when Unity iOS export has no Podfile
EDM4U + com.onesignal.unity.ios/Editor/OneSignaliOSDependencies.xml always generate a Podfile during iOS export because OneSignalXCFramework is consumed via CocoaPods. The previous fallback that silently skipped pod install on a missing Podfile would mask EDM4U / post-processor failures and ship an IPA without OneSignal native deps. Surface it as an ::error:: instead. Addresses Sherwin's review comment on #870. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent ca8a03f commit 81a5f76

1 file changed

Lines changed: 9 additions & 4 deletions

File tree

.github/workflows/e2e.yml

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -136,13 +136,18 @@ jobs:
136136
cacheUnityInstallationOnMac: true
137137

138138
- name: Pod install
139+
# EDM4U + com.onesignal.unity.ios/Editor/OneSignaliOSDependencies.xml
140+
# always generate a Podfile at iOS export time (OneSignalXCFramework is
141+
# consumed via CocoaPods). Missing Podfile means EDM4U or the SDK
142+
# post-processor silently failed; fail fast rather than ship an IPA
143+
# without OneSignal native deps.
139144
working-directory: examples/demo/Build/iOS
140145
run: |
141-
if [ -f Podfile ]; then
142-
pod install --repo-update
143-
else
144-
echo "No Podfile in Unity-exported Xcode project; skipping pod install"
146+
if [ ! -f Podfile ]; then
147+
echo "::error::No Podfile in Unity-exported Xcode project — EDM4U/OneSignal iOS post-processor likely failed"
148+
exit 1
145149
fi
150+
pod install --repo-update
146151
147152
- name: Set up iOS codesigning
148153
uses: OneSignal/sdk-shared/.github/actions/setup-ios-demo-codesigning@main

0 commit comments

Comments
 (0)