-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to docker-java 3.2.0 #2297
Conversation
# Conflicts: # build.gradle # core/src/main/java/org/testcontainers/dockerclient/DockerClientProviderStrategy.java
# Conflicts: # core/src/main/java/org/testcontainers/dockerclient/DockerClientConfigUtils.java # core/src/main/java/org/testcontainers/dockerclient/DockerClientProviderStrategy.java
/AzurePipelines run Windows 10 - Docker for Windows |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run Windows 10 - Docker for Windows |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run Windows 10 - Docker for Windows |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run Windows 10 - Docker for Windows |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run Windows 10 - Docker for Windows |
Azure Pipelines successfully started running 1 pipeline(s). |
@@ -520,12 +520,12 @@ void checksumFile(File file, Checksum checksum) { | |||
@SneakyThrows(JsonProcessingException.class) | |||
final String hash(CreateContainerCmd createCommand) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not consequential to the PR, but is there a reason for this method to be package-private? Should it be @VisibleForTesting
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC the idea was to make it protected at some point, but I don't really remember anymore 😅
/AzurePipelines run Windows 10 - Docker for Windows |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run Windows 10 - Docker for Windows |
Azure Pipelines successfully started running 1 pipeline(s). |
* Test new `docker-java` module * global Jitpack repo * update to the current master of docker-java * try docker-java-core * use 3.2.0-rc1 * update to docker-java 3.2.0-rc2 * remove unnecessary dependency * Update to docker-java 3.2.0-rc3, use their okhttp cmd factory * post-merge fixes * Update to docker-java 3.2.0-rc4 * fix JarFileShadingTest * Add japicmp task * Update to 3.2.0-rc5 * postpone shading changes * reduce the changeset * do not run japicmp on Windows * try running modules' tests without the checks * split module runs * fix the tasks list * fix the tasks list (2) * do not disable the checks * try `postCheckCommand` * make `postCheckCommand` optional * Update azure-pipelines.yml * Update to docker-java 3.2.0 * restore `postCheckCommand`
No description provided.