juju/series: remove dependency on the juju/testing package #3340

Merged
merged 1 commit into from Sep 21, 2015

Conversation

Projects
None yet
2 participants
Contributor

davecheney commented Sep 21, 2015

In preparation for moving juju/series to the utils repo, remove an
unneccessary dependency on github.com/juju/juju/testing. All the suites
in this package either need no base suite, or only require the basic
PatchValue/AddCleanup services offered by github.com/juju/testing.

(Review request: http://reviews.vapour.ws/r/2723/)

juju/series: remove dependency on the juju/testing package
In preparation for moving juju/series to the utils repo, remove an
unneccessary dependency on github.com/juju/juju/testing. All the suites
in this package either need no base suite, or only require the basic
PatchValue/AddCleanup services offered by github.com/juju/testing.
Contributor

davecheney commented Sep 21, 2015

$$merge$$

Contributor

jujubot commented Sep 21, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Sep 21, 2015

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/4816

Contributor

davecheney commented Sep 21, 2015

$$merge$$

Contributor

jujubot commented Sep 21, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Sep 21, 2015

Merge pull request #3340 from davecheney/clean-juju-series
juju/series: remove dependency on the juju/testing package

In preparation for moving juju/series to the utils repo, remove an
unneccessary dependency on github.com/juju/juju/testing. All the suites
in this package either need no base suite, or only require the basic
PatchValue/AddCleanup services offered by github.com/juju/testing.

(Review request: http://reviews.vapour.ws/r/2723/)

@jujubot jujubot merged commit dd3faf1 into juju:master Sep 21, 2015

@davecheney davecheney deleted the davecheney:clean-juju-series branch Sep 21, 2015

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