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

Fix type issue of the timelineEvents assignment #28530

Merged
merged 1 commit into from
Feb 27, 2019

Conversation

liyuqian
Copy link
Contributor

@liyuqian liyuqian commented Feb 26, 2019

This fixes #28374

The fact that json_rpc_2 is hiding exceptions like this might be a bigger issue: #28531

This will allow #27660 to reland to fix several very high priority performance issues.

This fixes flutter#28374

The fact that json_rpc_2 is hiding exceptions like this might be a
bigger issue: flutter#28531
Copy link
Member

@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

@liyuqian liyuqian merged commit 8e6f468 into flutter:master Feb 27, 2019
liyuqian added a commit that referenced this pull request Feb 27, 2019
This reverts commit adc8e15.

This should be safe to land once #28530 gets merged

Merge on yellow doc test because the doc test is actually green.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 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.

start_up tests in our post-submit bots seem to become flaky
3 participants