-
Notifications
You must be signed in to change notification settings - Fork 6k
[Impeller] 🔥 Cast fireball on host-debug-impeller-vulkan and see what breaks.
#45999
[Impeller] 🔥 Cast fireball on host-debug-impeller-vulkan and see what breaks.
#45999
Conversation
chinmaygarde
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The readme updates seem valuable regardless of what this attempt at removing the variant reveals. If we do back it out, we should perhaps split this patch up?
|
I actually didn't intend to reformat the markdown, but agreed the changes are good. |
bdero
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
auto label is removed for flutter/engine/45999, due to - The status or check suite Linux Web Framework tests has failed. Please fix the issues identified (or deflake) before re-applying this label. |
|
auto label is removed for flutter/engine/45999, due to - The status or check suite Linux Web Framework tests has failed. Please fix the issues identified (or deflake) before re-applying this label. |
|
Converting back to a draft PR. I want to try something else. |
|
This pull request has been changed to a draft. The currently pending flutter-gold status will not be able to resolve until a new commit is pushed or the change is marked ready for review again. |
host-debug-impeller-vulkan and see what breaks.host-debug-impeller-vulkan and see what breaks.
Closes flutter/flutter#134675.
This is based off @chinmaygarde's suggestion in flutter/flutter#134675 (comment).
@bdero could you review as current engine on-duty? Thanks!