Merge pull request #19 from dscho/some-fixups

Some fixups
git-for-windows · Jan 10, 2023 · 49fb949 · 49fb949
2 parents d769a86 + 389f4d8
commit 49fb949
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -0,0 +1,13 @@
+name: PR check
+
+on: [pull_request]
+
+jobs:
+  check:
+    runs-on: ubuntu-latest
+    steps:
+      - name: install ESLint
+        run: npm install -g eslint
+      - uses: actions/checkout@v3
+      - name: run ESLint
+        run: eslint -c .eslintrc.json '**/*.js'
diff --git a/GitForWindowsHelper/check-runs.js b/GitForWindowsHelper/check-runs.js
@@ -56,7 +56,6 @@ const queueCheckRun = async (context, token, owner, repo, ref, checkRunName, tit
       'PATCH',
       `/repos/${owner}/${repo}/check-runs/${checkRunId}`, 
       parameters
-      }
     )
   }
 

diff --git a/GitForWindowsHelper/slash-commands.js b/GitForWindowsHelper/slash-commands.js
@@ -133,7 +133,7 @@ module.exports = async (context, req) => {
 
             await thumbsUp()
 
-            const { queueCheckRun } = require('./check-runs')
+            const { queueCheckRun, updateCheckRun } = require('./check-runs')
             const triggerWorkflowDispatch = require('./trigger-workflow-dispatch')
             const triggerBuild = async (architecture) =>
                 await triggerWorkflowDispatch(
-Original file line number
+Diff line change
@@ Expand Up @@
           'PATCH',
           `/repos/${owner}/${repo}/check-runs/${checkRunId}`,
           parameters
-          }
         )
       }
@@ Expand Down @@