Merge branch 'master' into patch-1

dist/index.js

@@ -12806,7 +12806,10 @@ async function getPrFilesWithBlobSize(pullRequestNumber) {
             return !isExcluded;
         })
         : data;
-    const prFilesWithBlobSize = await Promise.all(files.map(async (file) => {
+    const prFilesWithBlobSize = await Promise.all(files
+        // Cannot get blobs for files without sha (e.g. happens when only changing a permission bit on the file)
+        .filter(file => file.sha != null)
+        .map(async (file) => {
         const { filename, sha, patch } = file;
         const { data: blob } = await octokit.rest.git.getBlob({
             ...repo,

package.json

@@ -30,7 +30,7 @@
   },
   "homepage": "https://github.com/ppremk/lfs-warning#readme",
   "dependencies": {
-    "@actions/core": "^1.5.0",
+    "@actions/core": "^1.9.1",
     "@actions/github": "^5.0.0",
     "micromatch": "^4.0.4"
   },

src/index.ts

@@ -198,21 +198,24 @@ async function getPrFilesWithBlobSize(pullRequestNumber: number) {
       : data;
 
   const prFilesWithBlobSize = await Promise.all(
-    files.map(async file => {
-      const {filename, sha, patch} = file;
-      const {data: blob} = await octokit.rest.git.getBlob({
-        ...repo,
-        file_sha: sha,
-      });
+    files
+      // Cannot get blobs for files without sha (e.g. happens when only changing a permission bit on the file)
+      .filter(file => file.sha != null)
+      .map(async file => {
+        const {filename, sha, patch} = file;
+        const {data: blob} = await octokit.rest.git.getBlob({
+          ...repo,
+          file_sha: sha,
+        });
 
-      return {
-        filename,
-        filesha: sha,
-        fileblobsize: blob.size,
-        filecontents: blob.content,
-        patch,
-      };
-    })
+        return {
+          filename,
+          filesha: sha,
+          fileblobsize: blob.size,
+          filecontents: blob.content,
+          patch,
+        };
+      })
   );
   return prFilesWithBlobSize;
 }