Skip to content

ci(repo): Reduce iOS simulator matrix in cron-checks to avoid queues on CI - #3066

Merged
testableapple merged 3 commits into
masterfrom
ci/matrix
Oct 15, 2026
Merged

testableapple merged 3 commits into
masterfrom
ci/matrix

Conversation

@testableapple

@testableapple testableapple commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Tests
    • The nightly iOS test suite now runs on iOS 26.2 using iPhone 17 Pro instead of iOS 18.5 on iPhone 16 Pro.
    • Manually triggered E2E workflows no longer run iOS tests by default; select the iOS option to run them.
    • Pull-request builds no longer include an iOS build.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 7fd20c8a-b10f-4fa1-b0a6-4e6ca6c55100
📥 Commits

Reviewing files that changed from the base of the PR and between a80cb31 and 6230681.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 31d2ade0-5e2e-4be4-abcd-57ea24dffffd
📥 Commits

Reviewing files that changed from the base of the PR and between 31301cb and a80cb31.

📒 Files selected for processing: 2
  • .github/workflows/e2e_test.yml
  • .github/workflows/stream_flutter_workflow.yml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

📝 Walkthrough

The E2E workflows update the nightly iOS test matrix and make iOS testing opt-in for manual dispatch. The Flutter pull-request build matrix no longer schedules an iOS build.

Changes: iOS workflow settings

Layer / File(s) Summary
Update iOS E2E settings
.github/workflows/e2e_test_cron.yml, .github/workflows/e2e_test.yml
The nightly matrix replaces iOS 18.5 / iPhone 16 Pro with iOS 26.2 / iPhone 17 Pro and sets setup_runtime: false. The manual-dispatch ios input now defaults to false.
Remove iOS from the pull-request build matrix
.github/workflows/stream_flutter_workflow.yml
The iOS matrix entry is commented out, so pull-request builds no longer schedule an iOS build.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested reviewers: renefloor

Merge Risk: ⚪ Minimal · up to a80cb

The nightly workflow uses a runner image documented to provide its requested simulator, while manual iOS E2E is opt-in and Flutter’s iOS PR build is intentionally removed. No actionable merge-blocking risk is established.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title accurately describes the main change: reducing the iOS simulator matrix in scheduled CI checks to avoid queues. It does not mention the related manual-dispatch and pull-request workflow chan…
✨ Finishing Touches: 💡 1
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@xsahil03x xsahil03x changed the title Reduce iOS simulator matrix in cron-checks to avoid queues on CI ci(repo): Reduce iOS simulator matrix in cron-checks to avoid queues on CI Oct 15, 2026
@testableapple
testableapple enabled auto-merge (squash) October 15, 2026 11:35
@testableapple
testableapple merged commit 709f47d into master Oct 15, 2026
26 checks passed
@testableapple
testableapple deleted the ci/matrix branch October 15, 2026 11:47
@codecov

codecov Bot commented Oct 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.55%. Comparing base (311e666) to head (6230681).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3066   +/-   ##
=======================================
  Coverage   78.55%   78.55%           
=======================================
  Files         452      452           
  Lines       29439    29439           
=======================================
  Hits        23127    23127           
  Misses       6312     6312           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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