Skip to content
This repository has been archived by the owner on Dec 19, 2017. It is now read-only.

Update to use package:observable & latest package:observe #9

Merged
merged 3 commits into from
Oct 3, 2016
Merged

Update to use package:observable & latest package:observe #9

merged 3 commits into from
Oct 3, 2016

Conversation

tvolkert
Copy link
Contributor

@tvolkert tvolkert commented Oct 2, 2016

No description provided.

@tvolkert
Copy link
Contributor Author

tvolkert commented Oct 2, 2016

@jmesserly

@tvolkert tvolkert mentioned this pull request Oct 2, 2016
Open
kevmoo
kevmoo suggested changes Oct 2, 2016
View reviewed changes
Copy link
Contributor

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

Changelog updates, please

@tvolkert
Copy link
Contributor Author

tvolkert commented Oct 2, 2016

Done

jmesserly
jmesserly approved these changes Oct 3, 2016
View reviewed changes
Copy link
Contributor

@jmesserly jmesserly left a comment

Choose a reason for hiding this comment

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

LGTM 👍

kevmoo
kevmoo suggested changes Oct 3, 2016
View reviewed changes
Copy link
Contributor

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

Delete the unused code – no comments

@kevmoo kevmoo merged commit 7b1ed8f into dart-archive:master Oct 3, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kevmoo kevmoo kevmoo approved these changes

@jmesserly jmesserly jmesserly approved these changes

Assignees
No one assigned
Labels
cla: yes
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@tvolkert @kevmoo @jmesserly @googlebot