Skip to content

fix(ci): consolidate setup_simulator action to firebase-ios-sdk - #1887

Merged
ncooke3 merged 1 commit into
mainfrom
nc.ci.consolidate-setup-simulator
Oct 30, 2026
Merged

ncooke3 merged 1 commit into
mainfrom
nc.ci.consolidate-setup-simulator

Conversation

@ncooke3

@ncooke3 ncooke3 commented Oct 30, 2026

Copy link
Copy Markdown
Member

Follow-up to #1882, #1886, and firebase/firebase-ios-sdk#16974.

Now that upstream firebase/firebase-ios-sdk resolves scripts/setup_simulator.sh via $GITHUB_ACTION_PATH and includes the runtime/duplicate-simulator hardening, this PR:

  • Switches all 14 .github/workflows/*.yml workflows to reference firebase/firebase-ios-sdk/.github/actions/setup_simulator@e1e760d7ab3a985cd590f284ad632afc7d192924.
  • Removes the temporary local copies of .github/actions/setup_simulator/action.yml and scripts/setup_simulator.sh (along with the fallback invocation in scripts/test.sh).

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request removes the temporary local simulator setup script (scripts/setup_simulator.sh), the corresponding GitHub Actions composite action, and the conditional block in scripts/test.sh that automatically ran the simulator setup. There are no review comments, so I have no feedback to provide.

@ncooke3
ncooke3 merged commit 892ecac into main Oct 30, 2026
111 of 112 checks passed
@ncooke3
ncooke3 deleted the nc.ci.consolidate-setup-simulator branch October 30, 2026 21:13
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