Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@mdebbar
Copy link
Contributor

@mdebbar mdebbar commented Dec 3, 2021

Now that engine PRs are uploading images to Gold, and the flutter-gold check is being applied to them correctly, we don't need the flutter/golden repo anymore.

This PR stops using the flutter/goldens repo in the web engine.

Next step would be to archive the flutter/goldens repo.

@mdebbar mdebbar added the platform-web Code specifically for the web engine label Dec 3, 2021
@google-cla google-cla bot added the cla: yes label Dec 3, 2021
@mdebbar mdebbar force-pushed the remove_goldens_repo branch from d5ac907 to e880f35 Compare January 18, 2022 18:35
@mdebbar mdebbar requested a review from yjbanov January 19, 2022 20:16
@mdebbar mdebbar marked this pull request as ready for review January 19, 2022 20:16
Copy link
Contributor

@yjbanov yjbanov left a comment

Choose a reason for hiding this comment

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

lgtm

@mdebbar mdebbar merged commit 83cfdcc into flutter:main Jan 24, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 24, 2022
jason-simmons pushed a commit to flutter/flutter that referenced this pull request Jan 24, 2022
* 016f458 Roll Skia from 67f92df4b059 to b894b0594eee (1 revision) (flutter/engine#31019)

* 83cfdcc [web] Remove usage of flutter/goldens (flutter/engine#30116)
clocksmith pushed a commit to clocksmith/flutter that referenced this pull request Mar 8, 2022
* 016f458 Roll Skia from 67f92df4b059 to b894b0594eee (1 revision) (flutter/engine#31019)

* 83cfdcc [web] Remove usage of flutter/goldens (flutter/engine#30116)
@mdebbar mdebbar deleted the remove_goldens_repo branch January 17, 2023 18:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes platform-web Code specifically for the web engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants