Merge pull request #62 from andershagbard/annotations-exitcode

Use exit code 0 if report available

Author: graygilmore

dist/index.js

@@ -70,7 +70,6 @@ function getDiffFilter(
 
 export async function addAnnotations(
   reports: ThemeCheckReport[],
-  exitCode: number,
   configContent: string,
   ghToken: string,
   fileDiff: string[] | undefined,
@@ -195,7 +194,7 @@ export async function addAnnotations(
     ...ctx.repo,
     check_run_id: check.data.id,
     name: CHECK_NAME,
-    conclusion: exitCode > 0 ? 'failure' : 'success',
+    conclusion: errorCount > 0 ? 'failure' : 'success',
     output: {
       title: CHECK_NAME,
       summary: `${errorCount} error(s), ${warningCount} warning(s) found`,

src/addAnnotations.ts

@@ -22,7 +22,7 @@ async function run() {
   try {
     await installCli(version);
     if (ghToken) {
-      const [{ report, exitCode }, configContent, fileDiff] =
+      const [{ report }, configContent, fileDiff] =
         await Promise.all([
           runChecksJson(
             themeRoot,
@@ -35,12 +35,11 @@ async function run() {
         ]);
       await addAnnotations(
         report,
-        exitCode,
         configContent,
         ghToken,
         fileDiff,
       );
-      process.exit(exitCode);
+      process.exit(0);
     } else {
       const { exitCode } = await runChecksText(
         themeRoot,