Skip to content
This repository was archived by the owner on Apr 29, 2022. It is now read-only.

Conversation

@JsouLiang
Copy link
Contributor

@chinmaygarde
Copy link
Member

Converting this to a draft till the linked engine patch is reviewed. Thanks.

@chinmaygarde chinmaygarde marked this pull request as draft March 24, 2022 18:14
@JsouLiang JsouLiang force-pushed the Use-New-DlPathEffect-Object branch 4 times, most recently from a70333d to 0a9dfe6 Compare March 31, 2022 15:26
@JsouLiang JsouLiang force-pushed the Use-New-DlPathEffect-Object branch from 0a9dfe6 to 6b8c78d Compare April 15, 2022 03:21
@chinmaygarde chinmaygarde marked this pull request as ready for review April 21, 2022 20:07
Copy link
Contributor

@flar flar left a comment

Choose a reason for hiding this comment

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

LGTM

@JsouLiang JsouLiang force-pushed the Use-New-DlPathEffect-Object branch from 6b8c78d to ff1119d Compare April 22, 2022 02:17
@JsouLiang
Copy link
Contributor Author

JsouLiang commented Apr 22, 2022

cc @chinmaygarde , why the ci.yaml validation always failed?

@dnfield
Copy link
Contributor

dnfield commented Apr 22, 2022

The CI yaml check can be ignored for this repo. It's something that's getting added automatically but isn't set up for this repo and likely never will be (we'll be moving the contents to flutter/engine before much longer anyway).

@flar flar merged commit c97557b into flutter:main Apr 22, 2022
flar added a commit that referenced this pull request Apr 22, 2022
flar added a commit that referenced this pull request Apr 22, 2022
JsouLiang added a commit to JsouLiang/impeller that referenced this pull request Apr 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants