Starts v0.10.0 by mfulgo · Pull Request #37 · docker-java/docker-java

added 14 commits

July 31, 2014 16:23
The Command and Response objects should have names that follow the same
form. I've arbitrarily decided on VerbNoun. Since this is an
API-breaking change and we're in pre-1.0.0, I've bumped the minor
version from 9 to 10.
This moves the Response objects into the same package as their
associated commands, since they're so tightly coupled.
This changes the CopyFileFromContainer's impl method to not call
toString on the copyConfig that it uses in the POST body, instead
relying on the default JSON serializer and marked fields. This will
allow us to use a reflection-based toString.

This commit also creates a test around the use of the
CopyFileFromContainer command.
This cuts down on some of the boilerplate code. Since we're already
bringing Apache Commons Lang, we might as well use it.
Replacing more boilder-plate code with something that'll be easier to
maintain.
This is the first step toward simplifying or getting rid of the
CommitConfig object.
If you don't have any of your own containers, the InfoCmd integration
test can fail. This commit makes sure there's at least one container
present.
This effectively hides the config objects from the public API, as I
expect they'll either go away or get simplified.
The HostConfig is already a sub-class of the InspectContainerResponse
object.
Having a slightly less generic class name is probably a good thing...
DockerClient is now an immutable class.  The only way to set credentials
now is at construction of the DockerClient, using the DockerClientConfig
object.
Conflicts:
	src/main/java/com/github/dockerjava/client/command/CreateContainerCmd.java

marcuslinke added a commit that referenced this pull request

Aug 2, 2014

@marcuslinke

@mfulgo mfulgo deleted the for-upstream-v0.10.0 branch

August 3, 2014 17:25