Skip to content

Conversation

@polina-c
Copy link
Contributor

@polina-c polina-c commented Jun 21, 2023

The merge should wait for DevTools 2.25.0

@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Jun 21, 2023
@polina-c polina-c requested a review from jacob314 June 21, 2023 18:58
@polina-c polina-c changed the title Stop accepting DiagnosticsNode as input from Devtools. Stop accepting DiagnosticsNode as input from DevTools. Jun 21, 2023
Copy link
Contributor

@jacob314 jacob314 left a comment

Choose a reason for hiding this comment

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

lgtm

@polina-c
Copy link
Contributor Author

Tested Devtools > Inspector with this version of flutter:

Screen.Recording.2023-06-21.at.3.45.43.PM.mov

@github-actions github-actions bot removed the framework flutter/packages/flutter repository. See also f: labels. label Jun 28, 2023
@polina-c polina-c changed the title Stop accepting DiagnosticsNode as input from DevTools. Cleanup: stop accepting DiagnosticsNode as input from DevTools. Jun 28, 2023
@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Jul 3, 2023
@polina-c polina-c added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 4, 2023
@polina-c polina-c merged commit 5b12b74 into flutter:master Jul 4, 2023
@polina-c polina-c deleted the cleanup1 branch July 4, 2023 17:12
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 5, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 5, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jul 5, 2023
flutter/flutter@590ef2d...35085c3

2023-07-05 [email protected] Improve documentation for `ColorSheme.fromImageProvider` (flutter/flutter#129952)
2023-07-05 [email protected] Roll Packages from a03b900 to 7042079 (2 revisions) (flutter/flutter#130013)
2023-07-04 [email protected] Cleanup: stop accepting DiagnosticsNode as input from DevTools. (flutter/flutter#129302)

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
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 16, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 17, 2023
@polina-c polina-c added the a: leak tracking Issues and PRs related to memory leaks detected by leak_tracker label Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: leak tracking Issues and PRs related to memory leaks detected by leak_tracker autosubmit Merge PR when tree becomes green via auto submit App framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants