Skip to content

Conversation

@floitschG
Copy link
Contributor

The dart:async zone API is in the process of becoming strong-mode clean.
This patch fixes a breakage introduced by this change.

The change should have no semantic difference, even when running with the non-strong zone API.

@@ -1,5 +1,5 @@
name: test
version: 0.12.24+5
version: 0.12.24+6
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to update the minimum sdk version as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No. This change should maintain the same semantics.

@floitschG
Copy link
Contributor Author

I added a Changelog entry.

Copy link
Member

@grouma grouma left a comment

Choose a reason for hiding this comment

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

LGTM assuming Travis passes. I'll handle publishing if you want.

@keertip keertip merged commit 4b3e011 into master Sep 19, 2017
@kevmoo kevmoo deleted the zone.strong branch September 19, 2017 19:15
@nex3
Copy link
Member

nex3 commented Sep 21, 2017

@floitschG It looks like as of 2.0.0-dev.1.0, test is still experiencing a bunch of Zone-related errors (for example). Are there further changes needed?

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants