Skip to content

Remove bringup property from android_engine_flags_* tests#185685

Merged
auto-submit[bot] merged 1 commit into
masterfrom
camsim99-patch-1
Apr 28, 2026
Merged

Remove bringup property from android_engine_flags_* tests#185685
auto-submit[bot] merged 1 commit into
masterfrom
camsim99-patch-1

Conversation

@camsim99
Copy link
Copy Markdown
Contributor

@camsim99 camsim99 commented Apr 28, 2026

Removed 'bringup' property from android_engine_flags_* tests added in #182241 because they are not flaking.

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.

Removed 'bringup' property from Linux Android emu tasks.
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Apr 28, 2026
@camsim99 camsim99 requested a review from jtmcdole April 28, 2026 17:19
@camsim99 camsim99 added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 28, 2026
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

Code Review

The pull request updates the .ci.yaml configuration by removing the bringup: true property from the Linux_android_emu android_engine_flags_debug_test and Linux_android_emu android_engine_flags_release_test targets. This change indicates that these tests are now considered stable and will block the CI pipeline upon failure. I have no feedback to provide.

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.

3 participants