Skip to content

[DO NOT MERGE] Trigger intentional golden test diff to test Skia Gold notifications - #191505

Open
andywolff wants to merge 1 commit into
flutter:masterfrom
andywolff:warn_goldens
Open

[DO NOT MERGE] Trigger intentional golden test diff to test Skia Gold notifications#191505
andywolff wants to merge 1 commit into
flutter:masterfrom
andywolff:warn_goldens

Conversation

@andywolff

@andywolff andywolff commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Exercises #191483

I changed the paint color in DlGoldenTest.CanDrawPaint from cyan to yellow to
produce an intentional golden image diff in impeller_golden_tests.

This PR tests whether the flutter-gold status check, the flutter-dashboard
triage comment, and the will affect goldens label are properly triggered on
CI presubmit runs.

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.

@github-actions github-actions Bot added engine flutter/engine related. See also e: labels. e: impeller Impeller rendering backend issues and features requests labels Aug 21, 2026
@andywolff andywolff changed the title [impeller] Trigger intentional golden test diff to test Skia Gold notifications [DO NOT MERGE] Trigger intentional golden test diff to test Skia Gold notifications Aug 21, 2026
@andywolff
andywolff marked this pull request as ready for review August 21, 2026 19:21
@andywolff andywolff added the CICD Run CI/CD label Aug 21, 2026

@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 CanDrawPaint golden unit test in dl_golden_unittests.cc, changing the paint color from cyan to yellow. I have no feedback to provide.

@andywolff

Copy link
Copy Markdown
Contributor Author

Confirmed that the desired golden warning was not attached to this PR. Looking at https://flutter-gold.skia.org/changelists to find my PR, I see the impacted goldens at https://flutter-gold.skia.org/search?issue=191505&crs=github&patchsets=1&corpus=flutter

I merged a fix to cocoon, so I'll try rebasing this PR after it goes live to verify the fixed behavior

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD e: impeller Impeller rendering backend issues and features requests engine flutter/engine related. See also e: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant