Sitelet https://github.com/flutter/flutter/pull/190775
Skip to content

Fix CocoaPods ruby dependency for flavors_test_ios on Mac bot - #190775

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
okorohelijah:fix_flavors_test_ruby
Aug 10, 2026
Merged

auto-submit[bot] merged 1 commit into
flutter:masterfrom
okorohelijah:fix_flavors_test_ruby

Conversation

@okorohelijah

@okorohelijah okorohelijah commented Aug 8, 2026 •

Copy link
Copy Markdown
Contributor

This PR adds the missing ruby (CocoaPods) dependency to the flavors_test_ios target in .ci.yaml.
Following the migration of this test from the physical device pool (Mac_ios) to the generic Mac simulator pool (in PR #189442), the target began failing in post-submit bringup with the error CocoaPods not installed or not in valid state.
Because the dummy flavors app uses the integration_test package and contains an ios/Podfile, it requires CocoaPods to compile. Generic Mac CI bots do not have CocoaPods installed globally, so it must be explicitly requested in the LUCI .ci.yaml dependencies list.

List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.

Fixes #189442

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

If this change needs to override an active code freeze, provide a comment explaining why. The code freeze workflow can be overridden by code reviewers. See pinned issues for any active code freezes with guidance.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

@okorohelijah
okorohelijah requested a review from vashworth August 8, 2026 03:49
@okorohelijah
okorohelijah marked this pull request as ready for review August 8, 2026 03:49

@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 updates the .ci.yaml configuration file to add a Ruby dependency (ruby_3.1-pod_1.13) to the flavors_test_ios target. There are no review comments, and I have no feedback to provide.

@okorohelijah okorohelijah added autosubmit Merge PR when tree becomes green via auto submit App CICD Run CI/CD labels Aug 10, 2026
@auto-submit
auto-submit Bot added this pull request to the merge queue Aug 10, 2026
Merged via the queue into flutter:master with commit c9c1537 Aug 10, 2026
39 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants