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

Conversation

@matanlurey
Copy link
Contributor

@jonahwilliams
Copy link
Contributor

Something seems up with CI

@matanlurey
Copy link
Contributor Author

I guess I broke the YAML format. Will fix it tomorrow 🙂

@matanlurey
Copy link
Contributor Author

I guess I broke the YAML format. Will fix it tomorrow 🙂

Or not, it was easy

@matanlurey matanlurey changed the title Remove --enable-impeller-{backend} flags that default to True. Remove --enable-impeller-{backend} flags (always True). Mar 14, 2024
Copy link
Contributor

@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

@matanlurey matanlurey merged commit 1da1597 into flutter:main Mar 14, 2024
@matanlurey matanlurey deleted the remove-default-true-impeller-be-flags branch March 14, 2024 05:13
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 14, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Mar 14, 2024
flutter/engine@bdf66af...1da1597

2024-03-14 matanlurey@users.noreply.github.com Remove `--enable-impeller-{backend}` flags (always `True`). (flutter/engine#51402)

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 bdero@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
@chinmaygarde
Copy link
Contributor

Awesome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Impeller] delete --enable-impeller-vulkan and --enable-impeller-opengles

3 participants