Silence debug messages about some unexpected reasons.

Specifically, the ones about compiler-artifact and build-script-executed, which aren't actually unexpected. Also adjust prettier print width to something a bit more reasonable.
10XGenomics · Dec 5, 2022 · 025759f · 025759f
1 parent 7971dac
commit 025759f
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 7 deletions.
diff --git a/.prettierrc.js b/.prettierrc.js
@@ -1,7 +1,7 @@
 module.exports = {
     "arrowParens": "always",
     "bracketSpacing": true,
-    "printWidth": 200,
+    "printWidth": 100,
     "quoteProps": "as-needed",
     "semi": true,
     "singleQuote": false,

diff --git a/dist/index.js b/dist/index.js
diff --git a/src/outputParser.ts b/src/outputParser.ts
@@ -40,7 +40,12 @@ export class OutputParser {
         }
 
         if (contents.reason !== "compiler-message") {
-            core.debug(`Unexpected reason field, ignoring it: ${contents.reason}`);
+            if (
+                contents.reason !== "compiler-artifact" &&
+                contents.reason !== "build-script-executed"
+            ) {
+                core.debug(`Unexpected reason field, ignoring it: ${contents.reason}`);
+            }
             return;
         }
 

diff --git a/src/reporter.ts b/src/reporter.ts
@@ -4,7 +4,11 @@ import type { AnnotationWithMessageAndLevel, Context, Stats } from "./schema";
 import { AnnotationLevel } from "./schema";
 
 export class Reporter {
-    public async report(stats: Stats, annotations: AnnotationWithMessageAndLevel[], context: Context): Promise<void> {
+    public async report(
+        stats: Stats,
+        annotations: AnnotationWithMessageAndLevel[],
+        context: Context,
+    ): Promise<void> {
         for (const annotation of annotations) {
             switch (annotation.level) {
                 case AnnotationLevel.Error: {