Skip to content

Conversation

@christopherfujino
Copy link
Contributor

@christopherfujino christopherfujino commented Mar 17, 2023

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Mar 17, 2023
Copy link
Contributor

@annagrin annagrin left a comment

Choose a reason for hiding this comment

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

LGTM!

@christopherfujino
Copy link
Contributor Author

Copy link
Contributor

@gspencergoog gspencergoog left a comment

Choose a reason for hiding this comment

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

32384589-a60f0e74-c078-11e7-9bc1-e5b5287aea9d

@annagrin
Copy link
Contributor

annagrin commented Mar 17, 2023

@christopherfujino do we have a timeout somewhere for one test file not to exceed 30 minutes? I think I added too many tests:) Will split them up and try to re-land after this revert is in.

@gspencergoog gspencergoog merged commit 6992149 into master Mar 17, 2023
@gspencergoog gspencergoog deleted the revert-122776-annagrin/fix-hot-restart-flake branch March 17, 2023 05:40
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 17, 2023
@christopherfujino
Copy link
Contributor Author

@christopherfujino do we have a timeout somewhere for one test file not to exceed 30 minutes? I think I added too many tests:) Will split them up and try to re-land after this revert is in.

Ahh you're right, looks like even when this passes it's just under the timeout.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants