Skip to content

Conversation

@loic-sharma
Copy link
Member

The desktop team uses the a: desktop label to triage pull requests.

- '**/*animation*'

'a: desktop':
- '**/linux/*'
Copy link
Contributor

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/**/*'

Copy link
Member Author

@loic-sharma loic-sharma Jul 19, 2023

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

@loic-sharma loic-sharma added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 20, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jul 20, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Jul 20, 2023

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.

Copy link
Member

@cbracken cbracken left a 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.

@loic-sharma
Copy link
Member Author

@cbracken That was the engine repo's version of this pull request :)

@goderbauer goderbauer added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Jul 25, 2023
@loic-sharma loic-sharma added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 26, 2023
@auto-submit auto-submit bot merged commit 61fd11d into flutter:master Jul 26, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 27, 2023
tarrinneal pushed a commit to flutter/packages that referenced this pull request Jul 27, 2023
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App c: contributor-productivity Team-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants