Skip to content

Conversation

@shihaohong
Copy link
Contributor

@shihaohong shihaohong commented Jun 24, 2020

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

@shihaohong shihaohong requested review from goderbauer and renyou June 24, 2020 23:03
@fluttergithubbot fluttergithubbot added f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. labels Jun 24, 2020
Copy link
Contributor

@renyou renyou left a comment

Choose a reason for hiding this comment

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

LGTM

@flutter-github-sync
Copy link

Started Google testing for this PR

@shihaohong
Copy link
Contributor Author

@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.

Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@renyou
Copy link
Contributor

renyou commented Jun 24, 2020 via email

@fluttergithubbot fluttergithubbot merged commit 17cb865 into master Jun 25, 2020
@fluttergithubbot fluttergithubbot deleted the revert-59992-revert-58686-fix-zoom-page-transition-build branch June 25, 2020 00:03
@flutter-github-sync
Copy link

Google testing passed!

@shihaohong shihaohong changed the title Reland [PageTransitionsBuilder] Fix 'ZoomPageTransition' built more than once [PageTransitionsBuilder] Reland Fix 'ZoomPageTransition' built more than once Jun 25, 2020
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants