feat: proper exit statuses (#56)

defi-wonderland · Aug 21, 2024 · d8ce793 · d8ce793
1 parent a66753b
commit d8ce793
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@defi-wonderland/natspec-smells",
-  "version": "1.1.4",
+  "version": "1.1.5",
   "description": "Automatically identify missing or incomplete natspec",
   "homepage": "https://github.com/defi-wonderland/natspec-smells#readme",
   "repository": {

diff --git a/src/main.ts b/src/main.ts
@@ -22,16 +22,17 @@ import { Validator } from './validator';
 
   if (!warnings.length) {
     console.warn('No issues found');
-    return;
-  }
-
-  warnings.forEach(({ location, messages }) => {
-    console.warn(location);
-    messages.forEach((message) => {
-      console.warn(`  ${message}`);
+    return process.exit(0);
+  } else {
+    warnings.forEach(({ location, messages }) => {
+      console.warn(location);
+      messages.forEach((message) => {
+        console.warn(`  ${message}`);
+      });
+      console.warn();
     });
-    console.warn();
-  });
+    return process.exit(1);
+  }
 })().catch(console.error);
 
 function getArguments(): Config {