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

Reverts "Make flow/embedded_views.h compatible with .clang_tidy." #48130

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

auto-submit[bot]
Copy link
Contributor

@auto-submit auto-submit bot commented Nov 16, 2023

Reverts #47994
Initiated by: jonahwilliams
This change reverts the following previous change:
Original Description:
Work towards flutter/flutter#134969.

All changes were made automatically (i.e. with --fix).

@auto-submit auto-submit bot added the revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests. label Nov 16, 2023
Copy link
Member

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@auto-submit auto-submit bot merged commit d8fb1b1 into main Nov 16, 2023
@auto-submit auto-submit bot deleted the revert_ea425773743bfaf614368752cfab2f427ecacc4d branch November 16, 2023 17:18
matanlurey added a commit that referenced this pull request Nov 16, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 16, 2023
matanlurey added a commit that referenced this pull request Nov 16, 2023
…#48137)

Reverts #48130.

Now that the `runIf` clauses are omitted, CI should be a reliable source
of truth.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 16, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 16, 2023
…138568)

flutter/engine@22baa83...094a338

2023-11-16 [email protected] Roll Skia from 297049bbfc0d to 4c099aaa259f (5 revisions) (flutter/engine#48138)
2023-11-16 [email protected] Roll Dart SDK from b512191e9612 to 906f23c1cb20 (1 revision) (flutter/engine#48136)
2023-11-16 [email protected] Disable the `runIfNot` clauses in `.ci.yaml`, as they are unsafe. (flutter/engine#48132)
2023-11-16 [email protected] Fix race condition in Unobstructed Platform View Scenario tests (flutter/engine#48096)
2023-11-16 [email protected] [Impeller] store all path point data in single buffer. (flutter/engine#47896)
2023-11-16 [email protected] Roll Skia from b9ead4140f84 to 297049bbfc0d (2 revisions) (flutter/engine#48133)
2023-11-16 [email protected] [macOS] Replace pasteboard mock with fake (flutter/engine#48110)
2023-11-16 [email protected] [Impeller] Fix issue where the lock was not re-acquired when the wait exits on CV. (flutter/engine#48104)
2023-11-16 [email protected] [Impeller] Create a drawable backed TextureMTL. (flutter/engine#48049)
2023-11-16 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Make `flow/embedded_views.h` compatible with `.clang_tidy`." (flutter/engine#48130)
2023-11-16 [email protected] [macOS] Replace fixture subclasses with usings (flutter/engine#48111)
2023-11-16 [email protected] Make `flow/embedded_views.h` compatible with `.clang_tidy`. (flutter/engine#47994)
2023-11-16 [email protected] Make `flow/...` compatible with `.clang_tidy`. (flutter/engine#47995)

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 [email protected],[email protected],[email protected] 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
platform-android platform-fuchsia platform-ios revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants