Merge pull request #27 from actions/increase-default-query-interval

Increase default query interval and fix time exceed bug
actions · Mar 28, 2022 · 0e15778 · 0e15778
2 parents a68b5a4 + 12940a5
commit 0e15778
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 21 deletions.
diff --git a/action.yml b/action.yml
@@ -20,7 +20,7 @@ inputs:
   reporting_interval:
     description: 'Time in milliseconds between two deployment status report (default: 1 second)'
     required: false
-    default: "1000"
+    default: "5000"
 outputs:
   page_url:
     description: 'URL to deployed Github Pages'
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/pre/index.js b/pre/index.js
diff --git a/pre/index.js.map b/pre/index.js.map
diff --git a/src/deployment.js b/src/deployment.js
@@ -138,12 +138,13 @@ class Deployment {
             core.setFailed('Failed with status code: ' + res.status)
             break
           }
-        }
-        // Handle timeout
-        if (Date.now() - startTime >= timeout) {
-          core.info('Timeout reached, aborting!')
-          core.setFailed('Timeout reached, aborting!')
-          return
+
+          // Handle timeout
+          if (Date.now() - startTime >= timeout) {
+            core.info('Timeout reached, aborting!')
+            core.setFailed('Timeout reached, aborting!')
+            return
+          }
         }
       } catch (error) {
         core.setFailed(error)