Branch: refs/heads/master Home: https://github.com/jenkinsci/docker-plugin Commit: d53c671a8c2eb54b10347d5ac32228264d427dc0 https://github.com/jenkinsci/docker-plugin/commit/d53c671a8c2eb54b10347d5ac32228264d427dc0 Author: Peter Darton <pjdar...@users.noreply.github.com> 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 jenkinsci-commits+unsubscr...@googlegroups.com. 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.