Skip to content

Conversation

stuartmorgan-g
Copy link
Collaborator

FLEPlugin* was FlutterPlugin* in flutter/engine#9074
This updates all the runner and plugin code for that change.

FLEPlugin* was renamed FlutterPlugin*; this updates all the runner and
plugin code for that change.
Copy link

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@bmaczak
Copy link

bmaczak commented May 30, 2019

Hi all! The Travis build seems to be failing due to it not using the latest flutter. Do you think you can merge this?

@stuartmorgan-g
Copy link
Collaborator Author

The Travis build seems to be failing due to it not using the latest flutter

The Travis build failed (expectedly) at the time this PR was posted because the engine change linked in the description hadn't been rolled into Flutter.

Do you think you can merge this?

Now that the roll has happened, yes; that was the point of having this PR ready. Landing it before the roll happened would have made it impossible for anyone to build with the latest version of this repository (against any version of Flutter).

@stuartmorgan-g stuartmorgan-g merged commit 8f9c547 into google:master May 30, 2019
@stuartmorgan-g stuartmorgan-g deleted the plugin-api-rename-update branch May 30, 2019 12:40
@rodydavis
Copy link

Thank you!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 8, 2021
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