Fix: don't use internal APIs of ESLint (fixes #171)

circle.yml

@@ -4,12 +4,12 @@ machine:
 
 dependencies:
   pre:
-    - nvm install 7
+    - nvm install 6
     - nvm install 8
 
 test:
   override:
-    - npm test
+    - nvm use 4 && npm test
     - nvm use 6 && npm test
-    - nvm use 7 && npm test
     - nvm use 8 && npm test
+    - nvm use 8 && npm i [email protected] --no-save && npm run -s test:base

lib/rules/attribute-hyphenation.js

@@ -43,7 +43,7 @@ function create (context) {
   // Public
   // ----------------------------------------------------------------------
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     VAttribute (node) {
       if (!utils.isCustomComponent(node.parent.parent)) return
 
@@ -53,8 +53,6 @@ function create (context) {
       reportIssue(node, name)
     }
   })
-
-  return {}
 }
 
 module.exports = {

lib/rules/html-end-tags.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     VElement (node) {
       const name = node.name
       const isVoid = utils.isHtmlVoidElementName(name)
@@ -48,8 +48,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/html-no-self-closing.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VElement' (node) {
       if (utils.isSvgElementNode(node)) {
         return
@@ -42,8 +42,6 @@ function create (context) {
       })
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/html-quotes.js

@@ -27,7 +27,7 @@ function create (context) {
   const quoteChar = double ? '"' : "'"
   const quoteName = double ? 'double quotes' : 'single quotes'
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VAttribute[value!=null]' (node) {
       const text = sourceCode.getText(node.value)
       const firstChar = text[0]
@@ -41,8 +41,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/html-self-closing.js

@@ -89,7 +89,7 @@ function create (context) {
   const sourceCode = context.getSourceCode()
   const options = parseOptions(context.options[0])
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VElement' (node) {
       const elementType = getElementType(node)
       const mode = options[elementType]
@@ -132,8 +132,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/max-attributes-per-line.js

@@ -71,7 +71,7 @@ module.exports = {
     const singlelinemMaximum = configuration.singleline
     const canHaveFirstLine = configuration.allowFirstLine
 
-    utils.registerTemplateBodyVisitor(context, {
+    return utils.defineTemplateBodyVisitor(context, {
       'VStartTag' (node) {
         const numberOfAttributes = node.attributes.length
 
@@ -155,7 +155,5 @@ module.exports = {
 
       return propsPerLine
     }
-
-    return {}
   }
 }

lib/rules/mustache-interpolation-spacing.js

@@ -65,7 +65,7 @@ module.exports = {
     // Public
     // ----------------------------------------------------------------------
 
-    utils.registerTemplateBodyVisitor(context, {
+    return utils.defineTemplateBodyVisitor(context, {
       'VExpressionContainer[expression!=null]' (node) {
         const tokens = template.getTokens(node, {
           includeComments: true,
@@ -85,7 +85,5 @@ module.exports = {
         }
       }
     })
-
-    return { }
   }
 }

lib/rules/no-confusing-v-for-v-if.js

@@ -37,7 +37,7 @@ function isUsingIterationVar (vIf) {
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='if']" (node) {
       const element = node.parent.parent
 
@@ -50,8 +50,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-duplicate-attributes.js

@@ -51,7 +51,7 @@ function create (context) {
     return directiveNames.has(name) || attributeNames.has(name)
   }
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VStartTag' () {
       directiveNames.clear()
       attributeNames.clear()
@@ -78,8 +78,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-bind.js

@@ -24,7 +24,7 @@ const VALID_MODIFIERS = new Set(['prop', 'camel', 'sync'])
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='bind']" (node) {
       for (const modifier of node.key.modifiers) {
         if (!VALID_MODIFIERS.has(modifier)) {
@@ -46,8 +46,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-cloak.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='cloak']" (node) {
       if (node.key.argument) {
         context.report({
@@ -47,8 +47,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-else-if.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='else-if']" (node) {
       const element = node.parent.parent
 
@@ -70,8 +70,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-else.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='else']" (node) {
       const element = node.parent.parent
 
@@ -70,8 +70,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-for.js

@@ -79,7 +79,7 @@ function checkKey (context, vFor, element) {
 function create (context) {
   const sourceCode = context.getSourceCode()
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='for']" (node) {
       const element = node.parent.parent
 
@@ -151,8 +151,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-html.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='html']" (node) {
       if (node.key.argument) {
         context.report({
@@ -47,8 +47,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-if.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='if']" (node) {
       const element = node.parent.parent
 
@@ -63,8 +63,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-model.js

@@ -80,7 +80,7 @@ function getVariable (name, leafNode) {
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='model']" (node) {
       const element = node.parent.parent
       const name = element.name
@@ -164,8 +164,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-on.js

@@ -31,7 +31,7 @@ const VERB_MODIFIERS = new Set([
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='on']" (node) {
       for (const modifier of node.key.modifiers) {
         if (!VALID_MODIFIERS.has(modifier) && !Number.isInteger(parseInt(modifier, 10))) {
@@ -52,8 +52,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-once.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='once']" (node) {
       if (node.key.argument) {
         context.report({
@@ -47,8 +47,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-pre.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='pre']" (node) {
       if (node.key.argument) {
         context.report({
@@ -47,8 +47,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-invalid-v-show.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='show']" (node) {
       if (node.key.argument) {
         context.report({
@@ -47,8 +47,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------