fix: better usage of simple git to assure asynchronous operations (#3)

derberg · Jan 22, 2021 · e2a6fcf · e2a6fcf
1 parent 7450f61
commit e2a6fcf
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 14 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -6107,13 +6107,12 @@ async function push(token, url, branchName, message, committerUsername, committe
 
   if (core.isDebug()) __webpack_require__(231).enable('simple-git');
 
-  return await git
-    .add('./*')
-    .addConfig('user.name', committerUsername)
-    .addConfig('user.email', committerEmail)
-    .commit(message)
-    .addRemote('auth', authanticatedUrl(token, url, committerUsername))
-    .push(['-u', 'auth', branchName]);
+  await git.add('./*');
+  await git.addConfig('user.name', committerUsername);
+  await git.addConfig('user.email', committerEmail);
+  await git.commit(message);
+  await git.addRemote('auth', authanticatedUrl(token, url, committerUsername));
+  await git.push(['-u', 'auth', branchName]);
 }
 
 

diff --git a/lib/git.js b/lib/git.js
@@ -20,12 +20,11 @@ async function push(token, url, branchName, message, committerUsername, committe
 
   if (core.isDebug()) require('debug').enable('simple-git');
 
-  return await git
-    .add('./*')
-    .addConfig('user.name', committerUsername)
-    .addConfig('user.email', committerEmail)
-    .commit(message)
-    .addRemote('auth', authanticatedUrl(token, url, committerUsername))
-    .push(['-u', 'auth', branchName]);
+  await git.add('./*');
+  await git.addConfig('user.name', committerUsername);
+  await git.addConfig('user.email', committerEmail);
+  await git.commit(message);
+  await git.addRemote('auth', authanticatedUrl(token, url, committerUsername));
+  await git.push(['-u', 'auth', branchName]);
 }