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

Conversation

@matanlurey
Copy link
Contributor

Towards flutter/flutter#152636.

Most of these were via dart --fix.

@matanlurey matanlurey requested a review from gaaclarke July 31, 2024 20:41
Copy link
Member

@gaaclarke gaaclarke left a comment

Choose a reason for hiding this comment

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

lgtm!

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 31, 2024
@auto-submit auto-submit bot merged commit 0fbff21 into flutter:main Jul 31, 2024
Copy link
Contributor

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

DBC

}

Picture CreateSimplePicture() {
Picture createSimplePicture() {
Copy link
Contributor

Choose a reason for hiding this comment

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

might could be private, too!

auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Aug 1, 2024
…152658)

flutter/engine@32f7888...0fbff21

2024-07-31 matanlurey@users.noreply.github.com Remove pre-existing diagnostics with no behavioral changes. (flutter/engine#54264)
2024-07-31 matanlurey@users.noreply.github.com Cleanup dart analyze diagnostics in ignored directories. (flutter/engine#54262)
2024-07-31 jason-simmons@users.noreply.github.com [web] Dispose a temporary image that may be created by instantiateImageCodecWithSize (flutter/engine#54096)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from b5ad5bf3696d to 8db309541ad9 (2 revisions) (flutter/engine#54266)
2024-07-31 skia-flutter-autoroll@skia.org Roll Dart SDK from 5df6a6e0c037 to efabc9163078 (1 revision) (flutter/engine#54263)

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://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
TytaniumDev pushed a commit to TytaniumDev/flutter that referenced this pull request Aug 7, 2024
…lutter#152658)

flutter/engine@32f7888...0fbff21

2024-07-31 matanlurey@users.noreply.github.com Remove pre-existing diagnostics with no behavioral changes. (flutter/engine#54264)
2024-07-31 matanlurey@users.noreply.github.com Cleanup dart analyze diagnostics in ignored directories. (flutter/engine#54262)
2024-07-31 jason-simmons@users.noreply.github.com [web] Dispose a temporary image that may be created by instantiateImageCodecWithSize (flutter/engine#54096)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from b5ad5bf3696d to 8db309541ad9 (2 revisions) (flutter/engine#54266)
2024-07-31 skia-flutter-autoroll@skia.org Roll Dart SDK from 5df6a6e0c037 to efabc9163078 (1 revision) (flutter/engine#54263)

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://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Buchimi pushed a commit to Buchimi/flutter that referenced this pull request Sep 2, 2024
…lutter#152658)

flutter/engine@32f7888...0fbff21

2024-07-31 matanlurey@users.noreply.github.com Remove pre-existing diagnostics with no behavioral changes. (flutter/engine#54264)
2024-07-31 matanlurey@users.noreply.github.com Cleanup dart analyze diagnostics in ignored directories. (flutter/engine#54262)
2024-07-31 jason-simmons@users.noreply.github.com [web] Dispose a temporary image that may be created by instantiateImageCodecWithSize (flutter/engine#54096)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from b5ad5bf3696d to 8db309541ad9 (2 revisions) (flutter/engine#54266)
2024-07-31 skia-flutter-autoroll@skia.org Roll Dart SDK from 5df6a6e0c037 to efabc9163078 (1 revision) (flutter/engine#54263)

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://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants