Skip to content

ci: fix build by adding pod install and pipefail#61

Open
kinyoklion wants to merge 2 commits into
mainfrom
devin/1781300465-fix-ci-pipefail
Open

ci: fix build by adding pod install and pipefail#61
kinyoklion wants to merge 2 commits into
mainfrom
devin/1781300465-fix-ci-pipefail

Conversation

@kinyoklion

@kinyoklion kinyoklion commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

The CI build was silently failing for two reasons:

  1. Missing pod install: When the vendored Pods/ directory was removed (commit 9a0eedb), a pod install step was never added to CI. The xcworkspace references Pods/Pods.xcodeproj which doesn't exist without it.

  2. Missing pipefail: Piping xcodebuild to xcpretty without set -o pipefail meant the pipeline exit code came from xcpretty (always 0), masking the build failure.

Fixes both issues so CI correctly installs dependencies and reports failures.

Link to Devin session: https://app.devin.ai/sessions/5b1fe090ca3247c9a9e594a426c3a8ef
Requested by: @kinyoklion

Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
@devin-ai-integration devin-ai-integration Bot changed the title ci: add pipefail to detect xcodebuild failures ci: fix build by adding pod install and pipefail Jun 12, 2026
@kinyoklion kinyoklion marked this pull request as ready for review June 12, 2026 22:22
@kinyoklion kinyoklion requested a review from a team as a code owner June 12, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants