-
Notifications
You must be signed in to change notification settings - Fork 6k
[labeler] Add engine workflow #42566
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's wait a bit until everything looks right in framework.
/cc @zanderso
Happy to wait. To add context for Zach, landing this in the engine is much safer than the framework. The framework has a lot of custom Dart code for handling labels, so it's been hard to migrate everything over. The engine side was |
Everything in the framework looks good to me. There's an edge case with labeling based on user that is no longer supported, but it sounds like that isn't used anywhere. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…128356) flutter/engine@4571695...ca49946 2023-06-06 chillers@google.com [labeler] Add engine workflow (flutter/engine#42566) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Please CC jacksongardner@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
flutter/flutter#126002
Pre-launch Checklist
///
).