Skip to content

github: auto-deploy CocoaPods on release#9955

Open
xseignard wants to merge 1 commit into
google:mainfrom
xseignard:cocoapods-auto-deploy
Open

github: auto-deploy CocoaPods on release#9955
xseignard wants to merge 1 commit into
google:mainfrom
xseignard:cocoapods-auto-deploy

Conversation

@xseignard
Copy link
Copy Markdown

@xseignard xseignard commented May 5, 2026

Summary

The CocoaPods deploy workflow was workflow_dispatch-only, so a new GitHub release never triggered it: the pod had to be published manually after every release. This PR wires the deploy into the release flow.

  • Adds a cocoapods-deploy job to .github/workflows/release.yml that needs: [build-ios] and runs on the release event (and on workflow_dispatch with platform=ios). This mirrors the existing sonatype-publish job, which hangs off build-android for the Maven publish.
  • Renames .github/workflows/cocopods-deploy.yml.github/workflows/cocoapods-manual-deploy.yml (also fixing the original cocopods typo) and reframes it as a manual fallback for re-runs. The job inside is renamed to cocoapods-manual-deploy to disambiguate from the in-release one.
  • Updates docs_src/src_mdbook/src/release/guide.md section 11 so the release guide describes CocoaPods as automatic, with the manual workflow documented as a fallback.

The dependency chain is correct: pod trunk push validates spec.source (https://github.com/google/filament/releases/download/${TAG}/filament-${TAG}-ios.tgz), and needs: [build-ios] guarantees the asset upload step has completed before the deploy runs.

The CocoaPods deploy workflow was workflow_dispatch-only, so a new
release never published the pod automatically. Fold the deploy into
release.yml as a job that needs build-ios, mirroring how
sonatype-publish hangs off build-android. The standalone workflow is
renamed to cocoapods-manual-deploy.yml and kept as a fallback for
re-runs. Release guide updated accordingly.
@google-cla
Copy link
Copy Markdown

google-cla Bot commented May 5, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in file name

jobs:
cocoapods-deploy:
name: cocoapods-deploy
cocoapods-manual-deploy:
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renamed for disambiguation with job in release.yml

cocoapods-deploy:
name: cocoapods-deploy
runs-on: macos-14
needs: [build-ios]
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build-ios job guarantees the asset upload step has completed before the Cocoapods deploy runs.

@xseignard
Copy link
Copy Markdown
Author

cc @bejado as it might interest you

@pixelflinger pixelflinger requested a review from bejado May 18, 2026 16:58
@bejado
Copy link
Copy Markdown
Member

bejado commented May 21, 2026

Thank you! This looks good to me so far. We're not doing a release this week, but I'll take a closer look and see if I can test it out with next week's release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants