-
Notifications
You must be signed in to change notification settings - Fork 29.7k
[PageTransitionsBuilder] Reland Fix 'ZoomPageTransition' built more than once #60245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PageTransitionsBuilder] Reland Fix 'ZoomPageTransition' built more than once #60245
Conversation
renyou
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
|
Started Google testing for this PR |
|
@renyou Unrelated to the PR itself, but does/should the |
goderbauer
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
|
Don't think so. Still working on it.
…On Wed, Jun 24, 2020, 7:27 PM Shi-Hao Hong ***@***.***> wrote:
@renyou <https://github.com/renyou> Unrelated to the PR itself, but
does/should the waiting for tree to go green label account for Google
testing for the PR? I was thinking of the case where this PR merges before
we get a result from Google testing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#60245 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFCW2QOATAUBCF7FOBGGYTRYKDVFANCNFSM4OHOLMFA>
.
|
|
Google testing passed! |
…lt more than once (flutter#58686)" (flutter#59992)" (flutter#60245)
Description
This has been unblocked internally via cl/318079801. b/159779895 was added to track fixing the tests that were originally preventing #58686 from landing.
cc/ @AlexVincent525
Issues
Fixes #58686
Reverts #59992