Fix pylint error at no-deprecated-router-link-tag-prop

vuejs · Oct 18, 2021 · ce5f19c · ce5f19c
1 parent be4e0e8
commit ce5f19c
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 20 deletions.
diff --git a/lib/rules/no-deprecated-router-link-tag-prop.js b/lib/rules/no-deprecated-router-link-tag-prop.js
@@ -21,7 +21,7 @@ module.exports = {
       description:
         'disallow using deprecated `tag` property on `RouterLink` (in Vue.js 3.0.0+)',
       categories: undefined,
-      url: 'https://eslint.vuejs.org/rules/no-deprecated-router-link-tag-prop'
+      url: 'https://eslint.vuejs.org/rules/no-deprecated-router-link-tag-prop.html'
     },
     fixable: null,
     schema: [
@@ -40,7 +40,8 @@ module.exports = {
       }
     ],
     messages: {
-      deprecated: "'tag' property on '{{element}}' component is deprecated. Use scoped slots instead."
+      deprecated:
+        "'tag' property on '{{element}}' component is deprecated. Use scoped slots instead."
     }
   },
   /** @param {RuleContext} context */
@@ -51,16 +52,16 @@ module.exports = {
     }
 
     return utils.defineTemplateBodyVisitor(context, {
-      "VElement"(node) {
+      VElement(node) {
         if (!components.includes(node.rawName)) return
 
         const attributes = node.startTag.attributes
-        attributes.forEach(attr => {
+        attributes.forEach((attr) => {
           /** @type VIdentifier | null */
-          let tagAttr = null;
+          let tagAttr = null
 
           if (attr.key.type === 'VIdentifier') {
-            tagAttr = attr.key;
+            tagAttr = attr.key
           } else if (attr.directive && attr.key.type === 'VDirectiveKey') {
             const arg = attr.key.argument
             if (arg && arg.type === 'VIdentifier') {

diff --git a/tests/lib/rules/no-deprecated-router-link-tag-prop.js b/tests/lib/rules/no-deprecated-router-link-tag-prop.js
@@ -52,7 +52,7 @@ tester.run('no-deprecated-router-link-tag-prop', rule, {
         </NuxtLink>
       </template>
       `
-    },
+    }
   ],
   invalid: [
     {
@@ -64,10 +64,11 @@ tester.run('no-deprecated-router-link-tag-prop', rule, {
       `,
       errors: [
         {
-          message: "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
+          message:
+            "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
           line: 3,
           column: 21
-        },
+        }
       ]
     },
     {
@@ -80,10 +81,11 @@ tester.run('no-deprecated-router-link-tag-prop', rule, {
       options: [{ components: ['RouterLink'] }],
       errors: [
         {
-          message: "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
+          message:
+            "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
           line: 3,
           column: 21
-        },
+        }
       ]
     },
     {
@@ -95,10 +97,11 @@ tester.run('no-deprecated-router-link-tag-prop', rule, {
       `,
       errors: [
         {
-          message: "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
+          message:
+            "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
           line: 3,
           column: 22
-        },
+        }
       ]
     },
     {
@@ -111,10 +114,11 @@ tester.run('no-deprecated-router-link-tag-prop', rule, {
       options: [{ components: ['RouterLink'] }],
       errors: [
         {
-          message: "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
+          message:
+            "'tag' property on 'RouterLink' component is deprecated. Use scoped slots instead.",
           line: 3,
           column: 22
-        },
+        }
       ]
     },
     {
@@ -127,10 +131,11 @@ tester.run('no-deprecated-router-link-tag-prop', rule, {
       options: [{ components: ['NuxtLink'] }],
       errors: [
         {
-          message: "'tag' property on 'NuxtLink' component is deprecated. Use scoped slots instead.",
+          message:
+            "'tag' property on 'NuxtLink' component is deprecated. Use scoped slots instead.",
           line: 3,
           column: 19
-        },
+        }
       ]
     },
     {
@@ -143,11 +148,12 @@ tester.run('no-deprecated-router-link-tag-prop', rule, {
       options: [{ components: ['NuxtLink'] }],
       errors: [
         {
-          message: "'tag' property on 'NuxtLink' component is deprecated. Use scoped slots instead.",
+          message:
+            "'tag' property on 'NuxtLink' component is deprecated. Use scoped slots instead.",
           line: 3,
           column: 20
-        },
+        }
       ]
-    },
+    }
   ]
 })