Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
worker: use external worker package #7070
Conversation
| @@ -11,10 +11,10 @@ import ( | ||
| "github.com/juju/testing" | ||
| jc "github.com/juju/testing/checkers" | ||
| gc "gopkg.in/check.v1" | ||
| + worker "gopkg.in/juju/worker.v1" |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
jujubot
merged commit c67da0e
into
juju:develop
Mar 8, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
rogpeppe commentedMar 6, 2017
The package gopkg.in/worker.v1 has existed for a while
and will soon have some changes that we need in Juju,
so use it instead of the locally defined Runner type
and delete that.
Also sort the imports canonically throughout.
Although large, there are no significant semantic
changes introduced by this PR.