Skip to content

Remove ios_debug_workflow test - #191141

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
vashworth:remove_ios_debug_workflow
Aug 18, 2026
Merged

auto-submit[bot] merged 1 commit into
flutter:masterfrom
vashworth:remove_ios_debug_workflow

Conversation

@vashworth

Copy link
Copy Markdown
Contributor

ios_debug_workflow was added to test lldb/devicectl workflow back when it was opt-in. It is now enabled by default so we can remove this test.

Fixes #191138.

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.

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Aug 14, 2026
@vashworth
vashworth requested a review from okorohelijah August 14, 2026 19:44

@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 ios_debug_workflow task from the devicelab. Specifically, it deletes the dev/devicelab/bin/tasks/ios_debug_workflow.dart test file, removes its entry from TESTOWNERS, and deletes the corresponding Mac_ios ios_debug_workflow target configuration in .ci.yaml. I have no feedback to provide.

@jtmcdole jtmcdole left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

💵 The CodeFu LGTM Note

The LGTM exchange rates are currently trading at:

  • 1 LGTM = golden images match closely enough if I squint.
  • 2 LGTMs = I checked that this won't break the devicelab benchmarks
  • 3 LGTMs = I trust that this C++ pointer won't leak memory every frame.
  • 4 LGTMs = CI is green on the first try without a single retry or shard failure

💵 Value officially backed by the full faith, credit, and compute budget of the Flutter EngProd Infrastructure.

image_1786120554622963

@Piinks Piinks added platform-ios iOS applications specifically team-ios Owned by iOS platform team labels Aug 17, 2026
@vashworth vashworth added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 18, 2026
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 18, 2026
@auto-submit

auto-submit Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/191141, because - The status or check suite Dashboard Checks has failed. Please fix the issues identified (or deflake) before re-applying this label.

@vashworth vashworth added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 18, 2026
@auto-submit
auto-submit Bot added this pull request to the merge queue Aug 18, 2026
Merged via the queue into flutter:master with commit 5f9e246 Aug 18, 2026
43 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 18, 2026
@vashworth vashworth mentioned this pull request Oct 29, 2026
2 of 5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD platform-ios iOS applications specifically team-ios Owned by iOS platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mac_ios ios_debug_workflow failing after https://github.com/flutter/flutter/pull/189993

4 participants