Branch: refs/heads/master
Home: https://github.com/jenkinsci/docker-plugin
Commit: d53c671a8c2eb54b10347d5ac32228264d427dc0
https://github.com/jenkinsci/docker-plugin/commit/d53c671a8c2eb54b10347d5ac32228264d427dc0
Author: Peter Darton <[email protected]>
Date: 2021-09-03 (Fri, 03 Sep 2021)
Changed paths:
M docs/attachments/docker-plugin-configuration-script.groovy
M src/main/java/com/nirima/jenkins/plugins/docker/DockerCloud.java
M
src/main/java/com/nirima/jenkins/plugins/docker/DockerContainerWatchdog.java
M
src/main/java/com/nirima/jenkins/plugins/docker/DockerImagePullStrategy.java
M src/main/java/com/nirima/jenkins/plugins/docker/DockerJobProperty.java
M src/main/java/com/nirima/jenkins/plugins/docker/DockerManagement.java
M
src/main/java/com/nirima/jenkins/plugins/docker/DockerManagementServer.java
M src/main/java/com/nirima/jenkins/plugins/docker/DockerRegistry.java
M src/main/java/com/nirima/jenkins/plugins/docker/DockerSimpleTemplate.java
M src/main/java/com/nirima/jenkins/plugins/docker/DockerTemplate.java
M src/main/java/com/nirima/jenkins/plugins/docker/DockerTemplateBase.java
M
src/main/java/com/nirima/jenkins/plugins/docker/action/DockerBuildAction.java
M
src/main/java/com/nirima/jenkins/plugins/docker/action/DockerBuildImageAction.java
M
src/main/java/com/nirima/jenkins/plugins/docker/builder/DockerBuilderControlCloudOption.java
M
src/main/java/com/nirima/jenkins/plugins/docker/builder/DockerBuilderControlOption.java
M
src/main/java/com/nirima/jenkins/plugins/docker/builder/DockerBuilderControlOptionDescriptor.java
M
src/main/java/com/nirima/jenkins/plugins/docker/builder/DockerBuilderControlOptionStopAll.java
M
src/main/java/com/nirima/jenkins/plugins/docker/builder/DockerBuilderNewTemplate.java
M
src/main/java/com/nirima/jenkins/plugins/docker/builder/DockerBuilderPublisher.java
R src/main/java/com/nirima/jenkins/plugins/docker/utils/DockerFunctions.java
M src/main/java/com/nirima/jenkins/plugins/docker/utils/JenkinsUtils.java
M src/main/java/com/nirima/jenkins/plugins/docker/utils/PortUtils.java
M src/main/java/io/jenkins/docker/DockerTransientNode.java
M src/main/java/io/jenkins/docker/FastNodeProvisionerStrategy.java
M src/main/java/io/jenkins/docker/client/DockerAPI.java
M
src/main/java/io/jenkins/docker/connector/DockerComputerAttachConnector.java
M src/main/java/io/jenkins/docker/connector/DockerComputerConnector.java
M src/main/java/io/jenkins/docker/connector/DockerComputerJNLPConnector.java
M src/main/java/io/jenkins/docker/connector/DockerComputerSSHConnector.java
M
src/main/java/io/jenkins/docker/connector/DockerDelegatingComputerLauncher.java
M src/main/java/io/jenkins/docker/pipeline/DockerNodeStep.java
M src/main/java/io/jenkins/docker/pipeline/DockerNodeStepExecution.java
M
src/test/java/com/nirima/jenkins/plugins/docker/DockerContainerWatchdogTest.java
M
src/test/java/com/nirima/jenkins/plugins/docker/TestableDockerContainerWatchdog.java
M src/test/java/com/nirima/jenkins/plugins/docker/utils/PortUtilsTest.java
M
src/test/java/com/nirima/jenkins/plugins/docker/utils/UniqueIdGeneratorTest.java
Log Message:
-----------
Code tidy (#851)
- Use Jenkins.get() in most places
- Use Jenkins.getInstanceOrNull() where null is ok during unit tests
- Rename local variables to stop hiding class fields, which caused warnings in
my IDE
- remove unused class
- remove unused import
- removed unnecessary else clauses
- where caught exceptions are fully handled by the code, declare then with
name "handledByCode"
- where caught exceptions are suppressed by the code, declare then with name
"ignored"
- fixed typo applyFoCloud -> applyToCloud
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/docker-plugin/push/refs/heads/master/1e8980-d53c67%40github.com.
