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.

Reply via email to