jujuclient: move MemStore from jujuclienttesting #7208

Merged
merged 1 commit into from Apr 7, 2017

Conversation

Projects
None yet
4 participants
Owner

rogpeppe commented Apr 6, 2017

The MemStore is useful for external programs that don't
want to use the disk-based store, and isn't directly
concerned with testing, so move it into the
jujuclient package.

This also means that test packages that want to use
MemStore don't need to depend on jujuclienttesting.

Also sort import statements canonically where they're
not.

This change is purely mechanical - no externally visible changes.

jujuclient: move MemStore from jujuclienttesting
The MemStore is useful for external programs that don't
want to use the disk-based store, and isn't directly
concerned with testing, so move it into the
jujuclient package.

This also means that test packages that want to use
MemStore don't need to depend on jujuclienttesting.

mhilton approved these changes Apr 6, 2017

LGTM

cmars approved these changes Apr 6, 2017

Owner

rogpeppe commented Apr 6, 2017

$$merge$$

Contributor

jujubot commented Apr 6, 2017

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

Contributor

jujubot commented Apr 6, 2017

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

Owner

rogpeppe commented Apr 7, 2017

$$merge$$

Contributor

jujubot commented Apr 7, 2017

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

@jujubot jujubot merged commit 69ce278 into juju:develop Apr 7, 2017

1 check failed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment