refactor(logs): simplify the syntax to colour the logs

actions · May 25, 2021 · 57a1997 · 57a1997
1 parent 3488a07
commit 57a1997
Show file tree

Hide file tree

Showing 9 changed files with 318 additions and 260 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -28,7 +28,7 @@
     "@typescript-eslint/no-array-constructor": "error",
     "@typescript-eslint/no-empty-interface": "error",
     "@typescript-eslint/no-explicit-any": "off",
-    "@typescript-eslint/no-extraneous-class": "error",
+    "@typescript-eslint/no-extraneous-class": "off",
     "@typescript-eslint/no-for-in-array": "error",
     "@typescript-eslint/no-inferrable-types": "error",
     "@typescript-eslint/no-misused-new": "error",

diff --git a/dist/index.js b/dist/index.js
diff --git a/src/classes/assignees.ts b/src/classes/assignees.ts
@@ -1,11 +1,11 @@
-import styles from 'ansi-styles';
 import deburr from 'lodash.deburr';
 import {Option} from '../enums/option';
 import {wordsToList} from '../functions/words-to-list';
 import {IAssignee} from '../interfaces/assignee';
 import {IIssuesProcessorOptions} from '../interfaces/issues-processor-options';
 import {Issue} from './issue';
 import {IssueLogger} from './loggers/issue-logger';
+import {LoggerService} from '../services/logger.service';
 
 type CleanAssignee = string;
 
@@ -34,7 +34,7 @@ export class Assignees {
 
     if (this._shouldExemptAllAssignees()) {
       this._issueLogger.info(
-        `${styles.white.open}└──${styles.white.close}`,
+        LoggerService.white('└──'),
         'Skipping this $$type because it has an exempt assignee'
       );
 
@@ -45,7 +45,7 @@ export class Assignees {
 
     if (exemptAssignees.length === 0) {
       this._issueLogger.info(
-        `${styles.white.open}├──${styles.white.close}`,
+        LoggerService.white('├──'),
         `No assignee option was specified to skip the stale process for this $$type`
       );
       this._logSkip();
@@ -54,10 +54,8 @@ export class Assignees {
     }
 
     this._issueLogger.info(
-      `${styles.white.open}├──${styles.white.close}`,
-      `Found ${styles.cyan.open}${exemptAssignees.length}${
-        styles.cyan.close
-      } assignee${
+      LoggerService.white('├──'),
+      `Found ${LoggerService.cyan(exemptAssignees.length)} assignee${
         exemptAssignees.length > 1 ? 's' : ''
       } that can exempt stale on this $$type`
     );
@@ -69,13 +67,13 @@ export class Assignees {
 
     if (!hasExemptAssignee) {
       this._issueLogger.info(
-        `${styles.white.open}├──${styles.white.close}`,
+        LoggerService.white('├──'),
         'No assignee on this $$type can exempt the stale process'
       );
       this._logSkip();
     } else {
       this._issueLogger.info(
-        `${styles.white.open}└──${styles.white.close}`,
+        LoggerService.white('└──'),
         'Skipping this $$type because it has an exempt assignee'
       );
     }
@@ -92,15 +90,15 @@ export class Assignees {
   private _getExemptIssueAssignees(): string[] {
     if (this._options.exemptIssueAssignees === '') {
       this._issueLogger.info(
-        `${styles.white.open}├──${styles.white.close}`,
+        LoggerService.white('├──'),
         `The option ${this._issueLogger.createOptionLink(
           Option.ExemptIssueAssignees
         )} is disabled. No specific assignee can skip the stale process for this $$type`
       );
 
       if (this._options.exemptAssignees === '') {
         this._issueLogger.info(
-          `${styles.white.open}├──${styles.white.close}`,
+          LoggerService.white('├──'),
           `The option ${this._issueLogger.createOptionLink(
             Option.ExemptAssignees
           )} is disabled. No specific assignee can skip the stale process for this $$type`
@@ -114,12 +112,10 @@ export class Assignees {
       );
 
       this._issueLogger.info(
-        `${styles.white.open}├──${styles.white.close}`,
+        LoggerService.white('├──'),
         `The option ${this._issueLogger.createOptionLink(
           Option.ExemptAssignees
-        )} is set. ${styles.cyan.open}${exemptAssignees.length}${
-          styles.cyan.close
-        } assignee${
+        )} is set. ${LoggerService.cyan(exemptAssignees.length)} assignee${
           exemptAssignees.length === 1 ? '' : 's'
         } can skip the stale process for this $$type`
       );
@@ -132,12 +128,10 @@ export class Assignees {
     );
 
     this._issueLogger.info(
-      `${styles.white.open}├──${styles.white.close}`,
+      LoggerService.white('├──'),
       `The option ${this._issueLogger.createOptionLink(
         Option.ExemptIssueAssignees
-      )} is set. ${styles.cyan.open}${exemptAssignees.length}${
-        styles.cyan.close
-      } assignee${
+      )} is set. ${LoggerService.cyan(exemptAssignees.length)} assignee${
         exemptAssignees.length === 1 ? '' : 's'
       } can skip the stale process for this $$type`
     );
@@ -148,15 +142,15 @@ export class Assignees {
   private _getExemptPullRequestAssignees(): string[] {
     if (this._options.exemptPrAssignees === '') {
       this._issueLogger.info(
-        `${styles.white.open}├──${styles.white.close}`,
+        LoggerService.white('├──'),
         `The option ${this._issueLogger.createOptionLink(
           Option.ExemptPrAssignees
         )} is disabled. No specific assignee can skip the stale process for this $$type`
       );
 
       if (this._options.exemptAssignees === '') {
         this._issueLogger.info(
-          `${styles.white.open}├──${styles.white.close}`,
+          LoggerService.white('├──'),
           `The option ${this._issueLogger.createOptionLink(
             Option.ExemptAssignees
           )} is disabled. No specific assignee can skip the stale process for this $$type`
@@ -170,12 +164,10 @@ export class Assignees {
       );
 
       this._issueLogger.info(
-        `${styles.white.open}├──${styles.white.close}`,
+        LoggerService.white('├──'),
         `The option ${this._issueLogger.createOptionLink(
           Option.ExemptAssignees
-        )} is set. ${styles.cyan.open}${exemptAssignees.length}${
-          styles.cyan.close
-        } assignee${
+        )} is set. ${LoggerService.cyan(exemptAssignees.length)} assignee${
           exemptAssignees.length === 1 ? '' : 's'
         } can skip the stale process for this $$type`
       );
@@ -188,12 +180,10 @@ export class Assignees {
     );
 
     this._issueLogger.info(
-      `${styles.white.open}├──${styles.white.close}`,
+      LoggerService.white('├──'),
       `The option ${this._issueLogger.createOptionLink(
         Option.ExemptPrAssignees
-      )} is set. ${styles.cyan.open}${exemptAssignees.length}${
-        styles.cyan.close
-      } assignee${
+      )} is set. ${LoggerService.cyan(exemptAssignees.length)} assignee${
         exemptAssignees.length === 1 ? '' : 's'
       } can skip the stale process for this $$type`
     );
@@ -211,7 +201,7 @@ export class Assignees {
 
         if (isSameAssignee) {
           this._issueLogger.info(
-            `${styles.white.open}├──${styles.white.close}`,
+            LoggerService.white('├──'),
             `@${issueAssignee.login} is assigned on this $$type and is an exempt assignee`
           );
         }
@@ -293,7 +283,7 @@ export class Assignees {
 
   private _logSkip(): void {
     this._issueLogger.info(
-      `${styles.white.open}└──${styles.white.close}`,
+      LoggerService.white('└──'),
       'Skip the assignees checks'
     );
   }