-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Add the 'a: desktop' label to labeler.yml #130933
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
.github/labeler.yml
Outdated
| - '**/*animation*' | ||
|
|
||
| 'a: desktop': | ||
| - '**/linux/*' |
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.
Should these have an additional directory wildcard at the end?
- '**/linux/**/*'
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.
That's what I thought too, but strangely the other paths don't seem to follow that pattern. For example, material definitely has nested folders, yet its pattern is `**/material/*. I'll add the additional directory wildcard
|
auto label is removed for flutter/flutter, pr: 130933, due to - The status or check suite Windows build_tests_4_4 has failed. Please fix the issues identified (or deflake) before re-applying this label. |
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.
I swear I remember lgtm-ing this, but maybe that was a different but related patch? Either way, lgtm!
Edit: Ah that was the engine side patch.
|
@cbracken That was the engine repo's version of this pull request :) |
Manual roll requested by [email protected] flutter/flutter@bae1ac2...61fd11d 2023-07-26 [email protected] Add the 'a: desktop' label to labeler.yml (flutter/flutter#130933) 2023-07-26 [email protected] ImageDecoration.lerp (#130533) (flutter/flutter#131349) 2023-07-26 [email protected] Minor adjustments on 2D APIs (flutter/flutter#131358) 2023-07-26 [email protected] Revert "Update Unit Tests for M2/M3" (flutter/flutter#131368) 2023-07-26 [email protected] Update Unit Tests for M2/M3 (flutter/flutter#131292) 2023-07-26 [email protected] Revert "Run benchmarks with `--omit-type-checks`" (flutter/flutter#131353) 2023-07-26 [email protected] Revert "ImageDecoration.lerp" (flutter/flutter#131347) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: 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
The desktop team uses the [`a: desktop`](https://github.com/flutter/engine/pulls?q=is%3Aopen+label%3A%22affects%3A+desktop%22+sort%3Aupdated-desc) label to [triage](https://github.com/flutter/flutter/wiki/Triage#desktop-platforms-team-team-desktop) pull requests.
The desktop team uses the [`a: desktop`](https://github.com/flutter/engine/pulls?q=is%3Aopen+label%3A%22affects%3A+desktop%22+sort%3Aupdated-desc) label to [triage](https://github.com/flutter/flutter/wiki/Triage#desktop-platforms-team-team-desktop) pull requests.
The desktop team uses the
a: desktoplabel to triage pull requests.