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

Reland elide long tree walks #49891

Merged
merged 6 commits into from Feb 1, 2020
Merged

Conversation

dnfield
Copy link
Contributor

@dnfield dnfield commented Jan 31, 2020

Relands #48413

Removes bad assertion from test (it isn't relevant to the test, forgot to remove it.

@fluttergithubbot fluttergithubbot added the framework flutter/packages/flutter repository. See also f: labels. label Jan 31, 2020
@shihaohong
Copy link
Contributor

It seems to still be failing the test for a different reason (doesn't seem to be eliding the tree walk?)

@shihaohong
Copy link
Contributor

shihaohong commented Jan 31, 2020

This seems to be failing now on packages/flutter/test/material/radio_test.dart: Radio ink ripple is displayed correctly for a framework test. Seems to be a Skia Gold error?

@dnfield
Copy link
Contributor Author

dnfield commented Jan 31, 2020

Both flakes - rerunning, and filed #49917

The other flake was a timeout during setup. Not sure if we have a bug tracking that, it's been happening more recently.

@Piinks
Copy link
Contributor

Piinks commented Jan 31, 2020

Could Cirrus be providing the wrong service account? IT looks like it is trying to authenticate with an account I haven't seen before.

@dnfield
Copy link
Contributor Author

dnfield commented Feb 1, 2020

CI all green now

Copy link
Contributor

@shihaohong shihaohong left a comment

Choose a reason for hiding this comment

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

LGTM

@fluttergithubbot fluttergithubbot merged commit a1aa3c5 into flutter:master Feb 1, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants