fix(#198): fix issue #198 by only displaying warn once

NullVoxPopuli · Nov 9, 2020 · 98a18a1 · 98a18a1
1 parent e5b6503
commit 98a18a1
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/lib/rules/decorator-position.js b/lib/rules/decorator-position.js
@@ -4,6 +4,9 @@ const { CLIEngine } = require('eslint');
 
 const cli = new CLIEngine();
 
+// currently used for incomplete prettier config
+let hasWarned = true;
+
 module.exports = {
   meta: {
     type: 'layout',
@@ -334,11 +337,13 @@ function lineLength(userOptions, filePath) {
 
   if (!('printWidth' in prettierOptions)) {
     // 80 is prettier default
-    // eslint-disable-next-line no-console
-    console.warn(
-      'WARNING(eslint-plugin-decorator-position): printWidth not set in the prettier config, defaulting to 80'
-    );
-
+    if (!hasWarned) {
+      // eslint-disable-next-line no-console
+      console.warn(
+        'WARNING(eslint-plugin-decorator-position): printWidth not set in the prettier config, defaulting to 80'
+      );
+      hasWarned = true;
+    }
     return { ...prettierOptions, ...userOptions, printWidth: 80 };
   }