Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
state: added State.WatchAllEnvs() #2977
Conversation
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$intermittent$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
$$broken-build-system$$ |
|
Build failed: Infrastructure failed |
|
$$broken-build-system-again$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$bad-record-mac$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
added a commit
that referenced
this pull request
Aug 14, 2015
mjs commentedAug 13, 2015
This includes the changes to Close() to clean up the allEnvWatcher plumbing.
Also added a missing unit test for State.Watch()
(Review request: http://reviews.vapour.ws/r/2357/)