Skip to content

Remove cocoapods trunk publishing - #131

Open
therama wants to merge 6 commits into
mainfrom
remove-cocoapods-trunk-publishing
Open

therama wants to merge 6 commits into
mainfrom
remove-cocoapods-trunk-publishing

Conversation

@therama

@therama therama commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Moves CocoaPods publishing exclusively to the OptiView Player specs repository, removing public trunk publishing while preserving dry-run validation.

Updates example Podfiles, installation guides, the main README, and changelog with the required specs source.

Validated with 15 stubbed workflow scenarios and syntax checks.

Validation covered the workflow’s command logic and file syntax:

  • 15 simulated scenarios: all seven connector choices (All, Utilities, Comscore, Conviva, Nielsen, SideloadedSubtitle, Uplynk) in both dry-run and release modes, plus one forced lint failure.
  • Publishing behavior: verified dry runs call pod spec lint without publishing, while releases configure the self-hosted repository and call pod repo push.
  • Dependency handling: checked source URLs, Nielsen’s extra source and validation flag, Utilities-first ordering, and the existing connector exclusions.
  • Cleanup: used temporary copies of the real podspecs to verify the tag-to-branch substitution and restoration after both successful validation and a forced failure.
  • Syntax and formatting: parsed the workflow YAML, checked the rendered shell script with bash -n, checked all four example Podfiles with ruby -c, and ran git diff --check.

The simulations stubbed pod and git commands. They did not perform actual CocoaPods dependency resolution, Xcode builds, authentication, or publishing. Those remain to be verified in CI.

@therama
therama requested review from Danesz, Dev1an and ceyhun-o November 3, 2026 16:02
fi
cmd="pod $command $connector $options"
if [ ${{ inputs.dryRun }} == false ]; then
cmd="$cmd --synchronous"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

aren't we missing this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That was a pod trunk push specific flag.

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