Skip to content
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

Flutter 1.22.0-12.1.pre engine cherrypicks #21188

Conversation

pcsosinski
Copy link

@flutter-dashboard
Copy link

This pull request was opened from and to a release candidate branch. This should only be done as part of the official Flutter release process. If you are attempting to make a regular contribution to the Flutter project, please close this PR and follow the instructions at Tree Hygiene for detailed instructions on contributing to Flutter.

Reviewers: Use caution before merging pull requests to release branches. Ensure the proper procedure has been followed.

Copy link
Member

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM regarding the commits. I took a look at @blasten 's commit and it looks ok, but it's non-trivial so he should also confirm.

Also, here's the license diff:

--- flutter/ci/licenses_golden/licenses_third_party	2020-09-15 19:23:29.109948078 +0000
+++ out/license_script_output/licenses_third_party	2020-09-15 19:36:39.671359357 +0000
@@ -1,7 +1,7 @@
-Signature: 14693ed982fa820c5dbce3c385be0a60
+Signature: a1bbcd05a2657658be7c5f38e0d366f4

Copy link

@blasten blasten left a comment

Choose a reason for hiding this comment

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

LGTM

@blasten
Copy link

blasten commented Sep 15, 2020

Thanks Patrick! LGTM

@pcsosinski pcsosinski merged commit 4654fc6 into flutter:flutter-1.22-candidate.12 Sep 15, 2020
@pcsosinski pcsosinski deleted the flutter-1.22-candidate.12 branch September 15, 2020 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants