[fixed] favor eslint-disable-next-line over disabling and then re-e…

…nabling a rule manually
thealjey · Apr 5, 2017 · 41c8c95 · 41c8c95
1 parent a30d433
commit 41c8c95
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 11 deletions.
diff --git a/src/JSLint.js b/src/JSLint.js
@@ -35,9 +35,8 @@ export class JSLint {
    */
   linter: CLIEngine;
 
-  /* eslint-disable require-jsdoc */
+  // eslint-disable-next-line require-jsdoc
   constructor(configFile: string = defaultConfigFile) {
-    /* eslint-enable require-jsdoc */
     this.linter = new CLIEngine({configFile});
   }
 

diff --git a/src/NativeProcess.js b/src/NativeProcess.js
@@ -42,9 +42,8 @@ export class NativeProcess {
    */
   proc: any;
 
-  /* eslint-disable require-jsdoc */
+  // eslint-disable-next-line require-jsdoc
   constructor(task: string) {
-    /* eslint-enable require-jsdoc */
     this.task = task;
   }
 

diff --git a/src/SASSLint.js b/src/SASSLint.js
@@ -36,9 +36,8 @@ export class SASSLint {
    */
   configFile: string;
 
-  /* eslint-disable require-jsdoc */
+  // eslint-disable-next-line require-jsdoc
   constructor(configFile: string = defaultConfigFile) {
-    /* eslint-enable require-jsdoc */
     this.configFile = configFile;
   }
 

diff --git a/src/highlight.js b/src/highlight.js
@@ -49,11 +49,9 @@ export function highlight(value: string): {dom: any; lines: any} {
   const dom = load(el.innerHTML),
     lines = dom('.CodeMirror-line');
 
-  /* eslint-disable lodash/prefer-lodash-method */
+  // eslint-disable-next-line lodash/prefer-lodash-method
   lines.find('> span').removeAttr('style');
 
-  /* eslint-enable lodash/prefer-lodash-method */
-
   return {dom, lines};
 }
 

diff --git a/src/logger.js b/src/logger.js
@@ -81,9 +81,8 @@ export class Message {
    */
   styles: string[] = [];
 
-  /* eslint-disable require-jsdoc */
+  // eslint-disable-next-line require-jsdoc
   constructor(style: ConsoleStyleConfig) {
-    /* eslint-enable require-jsdoc */
     this.style = style;
   }