Hopefully fix process exiting too early

lots0logs · Jun 5, 2020 · f1fbe31 · f1fbe31
1 parent 4367317
commit f1fbe31
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -367,7 +367,7 @@ function isRenamed(file) {
 	return 'renamed' === file.status;
 }
 
-function outputResults() {
+async function outputResults() {
 	debug('FILES', Array.from(FILES.values()));
 
 	core.setOutput('all', toJSON(Array.from(FILES.values()), 0));
@@ -387,7 +387,7 @@ function outputResults() {
 	fs.writeFileSync(`${process.env.HOME}/files_deleted.json`, toJSON(Array.from(FILES_REMOVED.values()), 0), 'utf-8');
 }
 
-function processCommitData(result) {
+async function processCommitData(result) {
 	debug('Processing API Response', result);
 
 	if (! result || ! result.data) {
@@ -444,9 +444,10 @@ getCommits().then(commits => {
 	}
 
 	Promise.all(commits.map(fetchCommitData))
-		.then(data => data.map(processCommitData))
+		.then(data => Promise.all(data.map(processCommitData)))
 		.then(outputResults)
-		.then(() => process.exit(0));
+		.then(() => process.exit(0))
+		.catch(err => core.error(err) && process.exit(1));
 });
 
 

diff --git a/index.js b/index.js
@@ -79,7 +79,7 @@ function isRenamed(file) {
 	return 'renamed' === file.status;
 }
 
-function outputResults() {
+async function outputResults() {
 	debug('FILES', Array.from(FILES.values()));
 
 	core.setOutput('all', toJSON(Array.from(FILES.values()), 0));
@@ -99,7 +99,7 @@ function outputResults() {
 	fs.writeFileSync(`${process.env.HOME}/files_deleted.json`, toJSON(Array.from(FILES_REMOVED.values()), 0), 'utf-8');
 }
 
-function processCommitData(result) {
+async function processCommitData(result) {
 	debug('Processing API Response', result);
 
 	if (! result || ! result.data) {
@@ -156,8 +156,9 @@ getCommits().then(commits => {
 	}
 
 	Promise.all(commits.map(fetchCommitData))
-		.then(data => data.map(processCommitData))
+		.then(data => Promise.all(data.map(processCommitData)))
 		.then(outputResults)
-		.then(() => process.exit(0));
+		.then(() => process.exit(0))
+		.catch(err => core.error(err) && process.exit(1));
 });
 
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "gh-action-get-changed-files",
-  "version": "2.1.1rc3",
+  "version": "2.1.1rc4",
   "private": true,
   "main": "dist/index.js",
   "scripts":{