Skip to content

Conversation

@cbracken
Copy link
Member

@cbracken cbracken commented Jun 13, 2018

fuchsia_tester.dart still assumes Dart 1. Previously, it ran tests directly
from source, flutter_platform.dart automatically runs a kernel compile when
operating in Dart 2 mode, but this assumes a functional Dart SDK is available
in the artifacts directly, and fuchsia_tester.dart mocks out the artifacts
directory with an empty temp dir.

Remaining work is:

  1. Get the frontend server building as a dependency on Fuchsia.
  2. Patch fuchsia_tester.dart to use a valid Dart SDK and frontend server.

This also reverts migration to Dart 2 typedef syntax.

This reverts commit 6c56bb2. (#18362)
This reverts commit 3daebd0. (#18316)

@cbracken cbracken merged commit 2ae4884 into flutter:master Jun 13, 2018
@cbracken cbracken deleted the its-back branch June 13, 2018 19:46
@cbracken
Copy link
Member Author

Unfixes #17051, #18278.

@Hixie
Copy link
Contributor

Hixie commented Jun 14, 2018

This was checked in when the tree was red. Was it expected that it would fix the tree?

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