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;
 }