Allow release for do_not_skip input (#273)

* Allow 'release' as option for 'do_not_skip' input
* Small optimizations
  * Use meaningful variable ('file' vs. 'f')
  * Remove unnecessary default values for boolean inputs
  * Directly return commit response (remove 'res' variable)

Author: paescuj

README.md

@@ -135,7 +135,7 @@ If true, skip if an already finished duplicate run can be found.
 
 A JSON-array with triggers that should never be skipped.
 
-Possible values are `pull_request`, `push`, `workflow_dispatch`, `schedule`.
+Possible values are `pull_request`, `push`, `workflow_dispatch`, `schedule`, `release`.
 
 **Default:** `'["workflow_dispatch", "schedule"]'`
 

dist/index.js

@@ -16,7 +16,8 @@ const workflowRunTriggerOptions = [
   'pull_request',
   'push',
   'workflow_dispatch',
-  'schedule'
+  'schedule',
+  'release'
 ] as const
 type WorkflowRunTrigger = typeof workflowRunTriggerOptions[number]
 
@@ -305,7 +306,9 @@ class SkipDuplicateActions {
       }
       iterSha = commit.parents?.length ? commit.parents[0]?.sha : null
       const changedFiles = commit.files
-        ? commit.files.map(f => f.filename).filter(f => typeof f === 'string')
+        ? commit.files
+            .map(file => file.filename)
+            .filter(file => typeof file === 'string')
         : []
       allChangedFiles.push(changedFiles)
 
@@ -417,11 +420,12 @@ class SkipDuplicateActions {
       return null
     }
     try {
-      const res = await this.context.octokit.rest.repos.getCommit({
-        ...this.context.repo,
-        ref: sha
-      })
-      return res.data
+      return (
+        await this.context.octokit.rest.repos.getCommit({
+          ...this.context.repo,
+          ref: sha
+        })
+      ).data
     } catch (error) {
       if (error instanceof Error || typeof error === 'string') {
         core.warning(error)
@@ -441,9 +445,10 @@ async function main(): Promise<void> {
     pathsFilter: getPathsFilterInput('paths_filter'),
     doNotSkip: getDoNotSkipInput('do_not_skip'),
     concurrentSkipping: getConcurrentSkippingInput('concurrent_skipping'),
-    cancelOthers: core.getBooleanInput('cancel_others') ?? false,
-    skipAfterSuccessfulDuplicates:
-      core.getBooleanInput('skip_after_successful_duplicate') ?? true
+    cancelOthers: core.getBooleanInput('cancel_others'),
+    skipAfterSuccessfulDuplicates: core.getBooleanInput(
+      'skip_after_successful_duplicate'
+    )
   }
 
   const repo = github.context.repo