refactor: remove error code logs for failure ones

Author: dastrong-codiga

ERROR_CODES.md

@@ -2,7 +2,7 @@ import fs from "fs";
 import YAML from "yaml";
 import { extname } from "path";
 import { ROSIE_SUPPORTED_SUFFIX_TO_LANGUAGE } from "./constants";
-import { printError } from "./print";
+import { printFailure } from "./print";
 
 /**
  * read a file contents
@@ -14,7 +14,7 @@ export function readFile(path) {
     const file = fs.readFileSync(path, "utf8");
     return file;
   } catch (err) {
-    printError(`unable to read file: ${path}`, "004");
+    printFailure(`Unable to read file: ${path}`);
     process.exit(1);
   }
 }
@@ -29,7 +29,7 @@ export function parseYamlFile(content, path) {
     const parsedFile = YAML.parse(content);
     return parsedFile;
   } catch (err) {
-    printError(`unable to parse YAML file${path ? `: ${path}` : ""}`, "005");
+    printFailure(`Unable to parse YAML file${path ? `: ${path}` : ""}`);
     process.exit(1);
   }
 }

utils/file.js

@@ -1,5 +1,5 @@
 import child_process from "child_process";
-import { printError } from "./print";
+import { printFailure } from "./print";
 
 /**
  * Executes a git command with the given args in that order
@@ -14,8 +14,6 @@ export function executeGitCommand(args) {
   } catch (err) {
     // console.debug(err)
     return null;
-    // printError("unable to execute a git command", "003");
-    // process.exit(2);
   }
 }
 
@@ -28,7 +26,9 @@ export function getRootDirectory() {
   if (rootDirectory) {
     return rootDirectory.split("\n").join("");
   } else {
-    printError("unable to execute a git command", "003");
+    printFailure(
+      "Unable to execute a git command because you're not in a git repository"
+    );
     process.exit(1);
   }
 }

utils/git.js

@@ -7,15 +7,6 @@ export function printEmptyLine() {
   console.log("");
 }
 
-/**
- * Used to print an error code and message for errors that need more explanation
- * @param {string} text
- * @param {string} code
- */
-export function printError(text, code) {
-  console.error(`%s: ${text}`, chalk.red.bold(`ERROR (${code})`));
-}
-
 /**
  * Used to indicate something negative and needs the user's attention
  * @param {string} text

utils/print.js

@@ -3,7 +3,7 @@ import { codigaApiFetch } from "./api";
 import { ACTION_TOKEN_ADD, CODIGA_CONFIG_FILE } from "./constants";
 import { getRootDirectory } from "./git";
 import { GET_RULESETS_FOR_CLIENT } from "../graphql/queries";
-import { printCommandSuggestion, printError } from "./print";
+import { printCommandSuggestion, printFailure } from "./print";
 
 /**
  * Gets an array of rulesets and their rules
@@ -16,7 +16,7 @@ export async function getRulesetsWithRules(names) {
     return rulesetsWithRules;
   } catch (err) {
     // console.debug(err);
-    printError("Unable to fetch your rulesets", "008");
+    printFailure("We were unable to fetch your rulesets");
     printCommandSuggestion(
       " ↳ Set a Codiga API token with one of the following commands:",
       ACTION_TOKEN_ADD
@@ -39,13 +39,13 @@ export function getRulesetsFromCodigaFile() {
 
   // if there isn't a rulesets value in the codiga.yml file, throw an error
   if (!parsedFile) {
-    printError("no rulesets value found in codiga.yml", "006");
+    printFailure("No rulesets value found in your `codiga.yml` file");
     process.exit(1);
   }
 
   // if there aren't any ruleset items under `rulesets:` in the codiga.yml file, throw an error
   if (!parsedFile.rulesets) {
-    printError("no rulesets found in codiga.yml", "007");
+    printFailure("No rulesets found in your `codiga.yml` file");
     process.exit(1);
   }