Fix commits with only filesToDelete

Author: mheap

create-or-update-files.js

@@ -82,7 +82,7 @@ module.exports = function (octokit, opts) {
       if (!baseTree || forkFromBaseBranch) {
         if (!createBranch && !baseTree) {
           return reject(
-            `The branch '${branchName}' doesn't exist and createBranch is 'false'`,
+            `The branch '${branchName}' doesn't exist and createBranch is 'false'`
           );
         }
 
@@ -125,7 +125,7 @@ module.exports = function (octokit, opts) {
 
         if (!hasFiles && !hasFilesToDelete) {
           return reject(
-            `either changes[].files or changes[].filesToDelete are required`,
+            `either changes[].files or changes[].filesToDelete are required`
           );
         }
 
@@ -140,14 +140,14 @@ module.exports = function (octokit, opts) {
                   owner,
                   repo,
                   fileName,
-                  baseTree,
+                  baseTree
                 );
 
                 // If it doesn't exist, and we're not ignoring missing files
                 // reject the promise
                 if (!exists && !change.ignoreDeletionFailures) {
                   return reject(
-                    `The file ${fileName} could not be found in the repo`,
+                    `The file ${fileName} could not be found in the repo`
                   );
                 }
 
@@ -160,40 +160,42 @@ module.exports = function (octokit, opts) {
                     type: "commit",
                   });
                 }
-              }),
+              })
             );
           }
         }
 
-        for (const batch of chunk(Object.keys(change.files), batchSize)) {
-          await Promise.all(
-            batch.map(async (fileName) => {
-              const properties = change.files[fileName] || "";
-
-              const contents = properties.contents || properties;
-              const mode = properties.mode || "100644";
-              const type = properties.type || "blob";
-
-              if (!contents) {
-                return reject(`No file contents provided for ${fileName}`);
-              }
-
-              const fileSha = await createBlob(
-                octokit,
-                owner,
-                repo,
-                contents,
-                type,
-              );
-
-              treeItems.push({
-                path: fileName,
-                sha: fileSha,
-                mode: mode,
-                type: type,
-              });
-            }),
-          );
+        if (hasFiles) {
+          for (const batch of chunk(Object.keys(change.files), batchSize)) {
+            await Promise.all(
+              batch.map(async (fileName) => {
+                const properties = change.files[fileName] || "";
+
+                const contents = properties.contents || properties;
+                const mode = properties.mode || "100644";
+                const type = properties.type || "blob";
+
+                if (!contents) {
+                  return reject(`No file contents provided for ${fileName}`);
+                }
+
+                const fileSha = await createBlob(
+                  octokit,
+                  owner,
+                  repo,
+                  contents,
+                  type
+                );
+
+                treeItems.push({
+                  path: fileName,
+                  sha: fileSha,
+                  mode: mode,
+                  type: type,
+                });
+              })
+            );
+          }
         }
 
         // no need to issue further requests if there are no updates, creations and deletions
@@ -207,7 +209,7 @@ module.exports = function (octokit, opts) {
           owner,
           repo,
           treeItems,
-          baseTree,
+          baseTree
         );
 
         // Create a commit that points to that tree
@@ -219,7 +221,7 @@ module.exports = function (octokit, opts) {
           author,
           message,
           tree,
-          baseTree,
+          baseTree
         );
 
         // Update the base tree if we have another commit to make
@@ -277,7 +279,7 @@ async function createCommit(
   author,
   message,
   tree,
-  baseTree,
+  baseTree
 ) {
   return (
     await octokit.rest.git.createCommit({