Skip to content

Conversation

@Hixie
Copy link
Contributor

@Hixie Hixie commented Feb 19, 2021

No description provided.

@google-cla google-cla bot added the cla: yes Contributor has signed the Contributor License Agreement label Feb 19, 2021
Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

I can merge this once you remove this from draft status and add it to the index file.

Thx!

@sfshaza2
Copy link
Contributor

@Hixie replied offline that it's not ready for merging.

@Hixie
Copy link
Contributor Author

Hixie commented Apr 23, 2021

Let's wait a few days to see if flutter/flutter#76288 gets reverted (there's a high risk of that happening), but if it sticks, this is ready...

@Hixie
Copy link
Contributor Author

Hixie commented Apr 26, 2021

It did; new PR is flutter/flutter#81235.

@Hixie
Copy link
Contributor Author

Hixie commented May 22, 2021

I need to update this to mention that onPlatformMessage is no longer called.

@Hixie
Copy link
Contributor Author

Hixie commented May 24, 2021

I should add something about tests that use package:test and package:flutter/service with test().

@Hixie
Copy link
Contributor Author

Hixie commented Jun 2, 2021

Ok I think this is ready to land!

@Hixie Hixie marked this pull request as ready for review June 2, 2021 23:52
Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

LGTM

@Hixie Hixie merged commit e5c19fb into master Jun 3, 2021
@Hixie Hixie deleted the Hixie-patch-2 branch June 3, 2021 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Contributor has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants