ci: use exit code instead of output text to detect pod trunk push success#51
Merged
andredestro merged 1 commit intomainfrom Apr 28, 2026
Merged
Conversation
jcesarmobile
approved these changes
Apr 28, 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.
Problem
The
publish-ios.shscript was incorrectly reporting a publish failure even whenpod trunk pushsucceeded. This happened because the success check relied on grepping for the word"passed"in the command output — a string that no longer appears in the current CocoaPods trunk response.As a result, the CI job exited with code 1 and printed an error, despite the pod being successfully published to CocoaPods.
Example of this happening: https://github.com/ionic-team/capacitor-camera/actions/runs/25047098127/job/73365335595
Root Cause
CocoaPods used to include
"passed"in its success output. The current output says"successfully published"instead, causing the grep to fail.Before — fragile: depends on CocoaPods output format:
Fix
Replace the text-based check with a direct exit code check, which is reliable regardless of CocoaPods output format changes.
After — reliable: checks the actual exit code: