Change default behaviour of "Suppress progress updates in job check"

src/main/java/io/jenkins/plugins/checks/github/GitHubChecksGlobalConfig.java

@@ -0,0 +1,46 @@
+package io.jenkins.plugins.checks.github;
+
+import hudson.Extension;
+import jenkins.model.GlobalConfiguration;
+import net.sf.json.JSONObject;
+import org.kohsuke.stapler.StaplerRequest;
+
+@Extension
+public class GitHubChecksGlobalConfig extends GlobalConfiguration {
+
+
+    private boolean skipProgressUpdates = false;
+    private boolean enforceSkipProgressUpdates = false;
+
+    public static GitHubChecksGlobalConfig get() {
+        return GlobalConfiguration.all().get(GitHubChecksGlobalConfig.class);
+    }
+
+    public GitHubChecksGlobalConfig() {
+        load();
+    }
+
+    public synchronized boolean isSkipProgressUpdates() {
+        return skipProgressUpdates;
+    }
+
+    public synchronized void setSkipProgressUpdates(boolean skipProgressUpdates) {
+        this.skipProgressUpdates = skipProgressUpdates;
+        save();
+    }
+
+    public synchronized boolean isEnforceSkipProgressUpdates() {
+        return enforceSkipProgressUpdates;
+    }
+
+    public synchronized void setEnforceSkipProgressUpdates(boolean enforceSkipProgressUpdates) {
+        this.enforceSkipProgressUpdates = enforceSkipProgressUpdates;
+        save();
+    }
+
+    @Override
+    public boolean configure(StaplerRequest req, JSONObject json) throws FormException {
+        req.bindJSON(this, json);
+        return true;
+    }
+}

src/main/java/io/jenkins/plugins/checks/github/status/GitHubSCMSourceStatusChecksTrait.java

@@ -1,5 +1,6 @@
 package io.jenkins.plugins.checks.github.status;
 
+import io.jenkins.plugins.checks.github.GitHubChecksGlobalConfig;
 import org.apache.commons.lang3.StringUtils;
 
 import org.kohsuke.stapler.DataBoundConstructor;
@@ -27,13 +28,17 @@ public class GitHubSCMSourceStatusChecksTrait extends SCMSourceTrait implements
     private boolean unstableBuildNeutral = false;
     private String name = "Jenkins";
     private boolean suppressLogs = false;
-    private boolean skipProgressUpdates = false;
+    private boolean skipProgressUpdates = DescriptorImpl.defaultSkipProgressUpdates;
+
+    public GitHubChecksGlobalConfig globalConfig = GitHubChecksGlobalConfig.get();
+    public boolean enforceSkipProgressUpdates = GitHubChecksGlobalConfig.get().isEnforceSkipProgressUpdates();
 
     /**
      * Constructor for stapler.
      */
     @DataBoundConstructor
     public GitHubSCMSourceStatusChecksTrait() {
+
         super();
     }
 
@@ -72,6 +77,9 @@ public boolean isSkipNotifications() {
         return skipNotifications;
     }
 
+    @Override
+    public boolean isEnforceSkipProgressUpdates() { return enforceSkipProgressUpdates; }
+
     @Override
     public boolean isSuppressLogs() {
         return suppressLogs;
@@ -136,6 +144,13 @@ protected void decorateContext(final SCMSourceContext<?, ?> context) {
      */
     @Extension
     public static class DescriptorImpl extends SCMSourceTraitDescriptor {
+        public static final boolean defaultSkipProgressUpdates;
+        public static final boolean enforceSkipProgressUpdates;
+
+        static {
+            enforceSkipProgressUpdates = GitHubChecksGlobalConfig.get().isEnforceSkipProgressUpdates();
+            defaultSkipProgressUpdates = true;
+        }
         /**
          * Returns the display name.
          *

src/main/java/io/jenkins/plugins/checks/github/status/GitHubStatusChecksConfigurations.java

@@ -1,5 +1,7 @@
 package io.jenkins.plugins.checks.github.status;
 
+import hudson.model.Job;
+
 /**
  * Configurations for users to customize status checks.
  */
@@ -39,6 +41,8 @@ public interface GitHubStatusChecksConfigurations {
      * @return true if progress updates should be skipped.
      */
     boolean isSkipProgressUpdates();
+
+    boolean isEnforceSkipProgressUpdates();
 }
 
 class DefaultGitHubStatusChecksConfigurations implements GitHubStatusChecksConfigurations {
@@ -66,5 +70,10 @@ public boolean isSuppressLogs() {
     public boolean isSkipProgressUpdates() {
         return false;
     }
+
+    @Override
+    public boolean isEnforceSkipProgressUpdates() {
+        return false;
+    }
 }
 

src/main/java/io/jenkins/plugins/checks/github/status/GitHubStatusChecksProperties.java

@@ -5,6 +5,7 @@
 
 import edu.hm.hafner.util.VisibleForTesting;
 
+import io.jenkins.plugins.checks.github.GitHubChecksGlobalConfig;
 import org.jenkinsci.plugins.github_branch_source.GitHubSCMSource;
 import hudson.Extension;
 import hudson.model.Job;
@@ -66,7 +67,10 @@ public boolean isSuppressLogs(final Job<?, ?> job) {
 
     @Override
     public boolean isSkipProgressUpdates(final Job<?, ?> job) {
-        return getConfigurations(job).orElse(DEFAULT_CONFIGURATION).isSkipProgressUpdates();
+        if (GitHubChecksGlobalConfig.get().isEnforceSkipProgressUpdates())
+            return GitHubChecksGlobalConfig.get().isSkipProgressUpdates();
+        else
+            return getConfigurations(job).orElse(DEFAULT_CONFIGURATION).isSkipProgressUpdates();
     }
 
     private Optional<GitHubStatusChecksConfigurations> getConfigurations(final Job<?, ?> job) {

src/main/java/io/jenkins/plugins/checks/github/status/GitSCMStatusChecksExtension.java

@@ -1,5 +1,6 @@
 package io.jenkins.plugins.checks.github.status;
 
+import io.jenkins.plugins.checks.github.GitHubChecksGlobalConfig;
 import org.apache.commons.lang3.StringUtils;
 
 import org.kohsuke.stapler.DataBoundConstructor;
@@ -22,6 +23,7 @@ public class GitSCMStatusChecksExtension extends GitSCMExtension implements GitH
     private String name = "Jenkins";
     private boolean suppressLogs;
     private boolean skipProgressUpdates = false;
+    public boolean enforceSkipProgressUpdates = GitHubChecksGlobalConfig.get().isEnforceSkipProgressUpdates();
 
     /**
      * Constructor for stapler.
@@ -56,6 +58,9 @@ public boolean isSkipProgressUpdates() {
         return skipProgressUpdates;
     }
 
+    @Override
+    public boolean isEnforceSkipProgressUpdates() { return enforceSkipProgressUpdates; }
+
     @DataBoundSetter
     public void setSkipProgressUpdates(final boolean skipProgressUpdates) {
         this.skipProgressUpdates = skipProgressUpdates;

src/main/resources/io/jenkins/plugins/checks/github/GitHubChecksGlobalConfig/config.jelly

@@ -0,0 +1,13 @@
+<?jelly escape-by-default='true'?>
+<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">
+    <f:section title="GitHub Checks Global Configuration">
+        <f:entry title="${%Suppress progress updates in job check}" field="skipProgressUpdates"
+                 description="${%Queued, checkout and completed will still be sent}">
+            <f:checkbox default="${descriptor.defaultSkipProgressUpdates}"/>
+        </f:entry>
+        <f:entry title="${%Enforce global setting suppress progress updates in job check}" field="enforceSkipProgressUpdates"
+                 description="${%Enforce this setting.}">
+            <f:checkbox/>
+        </f:entry>
+    </f:section>
+</j:jelly>

src/main/resources/lib/github-checks/status/properties.jelly

@@ -9,17 +9,19 @@
     <f:textbox default="Jenkins"/>
   </f:entry>
 
+
   <f:entry title="${%Publish unstable builds as neutral status checks}" field="unstableBuildNeutral">
     <f:checkbox/>
   </f:entry>
 
   <f:entry title="${%Suppress log output in checks}" field="suppressLogs">
     <f:checkbox/>
   </f:entry>
-
-  <f:entry title="${%Suppress progress updates in job check}" field="skipProgressUpdates"
-           description="${%Queued, checkout and completed will still be sent}">
-    <f:checkbox/>
-  </f:entry>
-
+  <j:var set="readOnlyMode" value="${descriptor.enforceSkipProgressUpdates}" />
+  <j:if test="${!readP}">
+    <f:entry title="${%Suppress progress updates in job check}" field="skipProgressUpdates"
+             description="${%Queued, checkout and completed will still be sent}">
+      <f:checkbox default="${descriptor.defaultSkipProgressUpdates}"/>
+    </f:entry>
+  </j:if>
 </j:jelly>