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

Conversation

@matanlurey
Copy link
Contributor

@matanlurey matanlurey commented Feb 15, 2024

The old code made me cry.

Potentially related to flutter/flutter#143471.

@johnmccutchan
Copy link
Contributor

love these kinds of cleanups

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 15, 2024
@flutter-dashboard
Copy link

Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change).

If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review.

Changes reported for pull request #50671 at sha 8b5fcd1

@matanlurey matanlurey merged commit 7130c35 into flutter:main Feb 15, 2024
@matanlurey matanlurey deleted the scenario-app_modern_locking branch February 15, 2024 01:51
sealesj pushed a commit to sealesj/engine that referenced this pull request Feb 15, 2024
The old code made me cry.

Potentially related to flutter/flutter#143471.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 16, 2024
jason-simmons added a commit to jason-simmons/flutter that referenced this pull request Feb 16, 2024
2024-02-15 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from mZP8LxbhYHstUxmxd... to fXqP_YC4zTp9G2hA5... (flutter/engine#50680)
2024-02-15 skia-flutter-autoroll@skia.org Roll Skia from b6201d9c2f73 to 85ab600a9519 (1 revision) (flutter/engine#50679)
2024-02-15 skia-flutter-autoroll@skia.org Roll Dart SDK from f1b1fa85a1be to 5e442dc80903 (1 revision) (flutter/engine#50678)
2024-02-15 chinmaygarde@google.com Use a GN variable (dart_src) to reference the location of the Dart checkout. (flutter/engine#50624)
2024-02-15 matanlurey@users.noreply.github.com Add a &right_filter=params to the Skia Gold links. (flutter/engine#50675)
2024-02-15 skia-flutter-autoroll@skia.org Roll Skia from 2d5cf67614d0 to b6201d9c2f73 (2 revisions) (flutter/engine#50674)
2024-02-15 matanlurey@users.noreply.github.com Disable Android API v28 tests on CI. (flutter/engine#50672)
2024-02-15 matanlurey@users.noreply.github.com Use modern concurrency constructs. (flutter/engine#50671)
2024-02-15 skia-flutter-autoroll@skia.org Roll Dart SDK from 322e34dc53f6 to f1b1fa85a1be (1 revision) (flutter/engine#50670)
2024-02-15 matanlurey@users.noreply.github.com Move boringssl to flutter/third_party. (flutter/engine#50601)
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Feb 16, 2024
2024-02-15 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from mZP8LxbhYHstUxmxd... to fXqP_YC4zTp9G2hA5... (flutter/engine#50680)
2024-02-15 skia-flutter-autoroll@skia.org Roll Skia from b6201d9c2f73 to 85ab600a9519 (1 revision) (flutter/engine#50679)
2024-02-15 skia-flutter-autoroll@skia.org Roll Dart SDK from f1b1fa85a1be to 5e442dc80903 (1 revision) (flutter/engine#50678)
2024-02-15 chinmaygarde@google.com Use a GN variable (dart_src) to reference the location of the Dart checkout. (flutter/engine#50624)
2024-02-15 matanlurey@users.noreply.github.com Add a &right_filter=params to the Skia Gold links. (flutter/engine#50675)
2024-02-15 skia-flutter-autoroll@skia.org Roll Skia from 2d5cf67614d0 to b6201d9c2f73 (2 revisions) (flutter/engine#50674)
2024-02-15 matanlurey@users.noreply.github.com Disable Android API v28 tests on CI. (flutter/engine#50672)
2024-02-15 matanlurey@users.noreply.github.com Use modern concurrency constructs. (flutter/engine#50671)
2024-02-15 skia-flutter-autoroll@skia.org Roll Dart SDK from 322e34dc53f6 to f1b1fa85a1be (1 revision) (flutter/engine#50670)
2024-02-15 matanlurey@users.noreply.github.com Move boringssl to flutter/third_party. (flutter/engine#50601)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App will affect goldens

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants