Skip to content

Conversation

@jonahwilliams
Copy link
Contributor

Reverts #39534

This is breaking the docs shard, see https://cirrus-ci.com/task/4755278756577280

The dart:wasm library was not added to the sky_engine rule, so the local dart sdk sources are in an inconsistent state (see flutter/engine@74a3c73)

@fluttergithubbot fluttergithubbot added the engine flutter/engine related. See also e: labels. label Aug 30, 2019
@jonahwilliams
Copy link
Contributor Author

TBR @dnfield @tvolkert

@jonahwilliams jonahwilliams merged commit 42550a1 into master Aug 30, 2019
@dnfield dnfield deleted the revert-39534-roll_branch branch August 30, 2019 03:32
@dnfield
Copy link
Contributor

dnfield commented Aug 30, 2019

LGTM.

@a-siva has dart added a new core lib for wasm?

@jonahwilliams
Copy link
Contributor Author

The fix is pending here: flutter/engine#11736

It was not added to the sky_engine rule, so dartdocs crashes trying to create docs for it

@tvolkert tvolkert self-requested a review August 30, 2019 03:44
@tvolkert
Copy link
Contributor

LGTM

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

engine flutter/engine related. See also e: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants