Add info into the meta property of rules

koddsson · koddsson · commit 7f54220417a7 · 2021-03-31T15:30:45.000+01:00
diff --git a/lib/rules/array-foreach.js b/lib/rules/array-foreach.js
@@ -1,6 +1,10 @@
 module.exports = {
   meta: {
-    docs: {},
+    type: 'suggestion',
+    docs: {
+      description: 'enforce `for..of` loops over `Array.forEach`',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/array-foreach.md'
+    },
     schema: []
   },
 
diff --git a/lib/rules/async-currenttarget.js b/lib/rules/async-currenttarget.js
@@ -1,6 +1,10 @@
 module.exports = {
   meta: {
-    docs: {},
+    type: 'problem',
+    docs: {
+      description: 'disallow `event.currentTarget` calls inside of async functions',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/async-currenttarget.md'
+    },
     schema: []
   },
 
diff --git a/lib/rules/async-preventdefault.js b/lib/rules/async-preventdefault.js
@@ -1,6 +1,10 @@
 module.exports = {
   meta: {
-    docs: {},
+    type: 'problem',
+    docs: {
+      description: 'disallow `event.preventDefault` calls inside of async functions',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/async-preventDefault.md'
+    },
     schema: []
   },
 
diff --git a/lib/rules/authenticity-token.js b/lib/rules/authenticity-token.js
@@ -1,6 +1,10 @@
 module.exports = {
   meta: {
-    docs: {},
+    type: 'problem',
+    docs: {
+      description: 'disallow usage of CSRF tokens in JavaScript',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/authenticity-token.md'
+    },
     schema: []
   },
 
diff --git a/lib/rules/get-attribute.js b/lib/rules/get-attribute.js
@@ -19,6 +19,15 @@ function isValidAttribute(name) {
 
 module.exports = function(context) {
   return {
+    meta: {
+      type: 'problem',
+      docs: {
+        description: 'disallow wrong usage of attribute names',
+        url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/get-attribute.md'
+      },
+      fixable: 'code',
+      schema: []
+    },
     CallExpression(node) {
       if (!node.callee.property) return
 
diff --git a/lib/rules/js-class-name.js b/lib/rules/js-class-name.js
@@ -1,6 +1,10 @@
 module.exports = {
   meta: {
-    docs: {},
+    type: 'suggestion',
+    docs: {
+      description: 'enforce a naming convention for js- prefixed classes',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/js-class-name.md'
+    },
     schema: []
   },
 
diff --git a/lib/rules/no-blur.js b/lib/rules/no-blur.js
@@ -1,11 +1,17 @@
 module.exports = function(context) {
   return {
+    meta: {
+      type: 'problem',
+      docs: {
+        description: 'disallow usage of `Element.prototype.blur()`',
+        url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-blur.md'
+      },
+      schema: []
+    },
     CallExpression(node) {
       if (node.callee.property && node.callee.property.name === 'blur') {
         context.report(node, 'Do not use element.blur(), instead restore the focus of a previous element.')
       }
     }
   }
 }
-
-module.exports.schema = []
diff --git a/lib/rules/no-d-none.js b/lib/rules/no-d-none.js
@@ -1,4 +1,12 @@
 module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'disallow usage the `d-none` CSS class',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-d-none.md'
+    },
+    schema: []
+  },
   create(context) {
     return {
       CallExpression(node) {
diff --git a/lib/rules/no-dataset.js b/lib/rules/no-dataset.js
@@ -1,6 +1,10 @@
 module.exports = {
   meta: {
-    docs: {},
+    type: 'problem',
+    docs: {
+      description: 'enforce usage of `Element.prototype.getAttribute` instead of `Element.prototype.datalist`',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-dataset.md'
+    },
     schema: []
   },
 
diff --git a/lib/rules/no-implicit-buggy-globals.js b/lib/rules/no-implicit-buggy-globals.js
@@ -1,6 +1,10 @@
 module.exports = {
   meta: {
-    docs: {},
+    type: 'problem',
+    docs: {
+      description: 'disallow implicit global variables',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-implicit-buggy-globals.md'
+    },
     schema: []
   },
 
diff --git a/lib/rules/no-innerText.js b/lib/rules/no-innerText.js
@@ -1,7 +1,12 @@
 module.exports = {
   meta: {
-    docs: {},
-    fixable: 'code'
+    type: 'problem',
+    docs: {
+      description: 'disallow `Element.prototype.innerText` in favor of `Element.prototype.textContent`',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-innerText.md'
+    },
+    fixable: 'code',
+    schema: []
   },
 
   create(context) {
diff --git a/lib/rules/no-then.js b/lib/rules/no-then.js
@@ -1,6 +1,11 @@
 module.exports = {
   meta: {
-    docs: {}
+    type: 'suggestion',
+    docs: {
+      description: 'enforce using `async/await` syntax over Promises',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-then.md'
+    },
+    schema: []
   },
 
   create(context) {
diff --git a/lib/rules/no-useless-passive.js b/lib/rules/no-useless-passive.js
@@ -4,8 +4,13 @@ const propIsPassiveTrue = prop => prop.key && prop.key.name === 'passive' && pro
 
 module.exports = {
   meta: {
-    docs: {},
-    fixable: 'code'
+    type: 'suggestion',
+    docs: {
+      description: 'disallow marking a event handler as passive when it has no effect',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-useless-passive.md'
+    },
+    fixable: 'code',
+    schema: []
   },
 
   create(context) {
diff --git a/lib/rules/prefer-observers.js b/lib/rules/prefer-observers.js
@@ -4,8 +4,12 @@ const observerMap = {
 }
 module.exports = {
   meta: {
-    docs: {},
-    fixable: 'code'
+    type: 'suggestion',
+    docs: {
+      description: 'disallow poorly performing event listeners',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/prefer-observers.md'
+    },
+    schema: []
   },
 
   create(context) {
diff --git a/lib/rules/require-passive-events.js b/lib/rules/require-passive-events.js
@@ -4,7 +4,12 @@ const propIsPassiveTrue = prop => prop.key && prop.key.name === 'passive' && pro
 
 module.exports = {
   meta: {
-    docs: {}
+    type: 'suggestion',
+    docs: {
+      description: 'enforce marking high frequency event handlers as passive',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/require-passive-events.md'
+    },
+    schema: []
   },
 
   create(context) {
diff --git a/lib/rules/unescaped-html-literal.js b/lib/rules/unescaped-html-literal.js
@@ -1,24 +1,35 @@
-module.exports = function(context) {
-  const htmlOpenTag = /^<[a-zA-Z]/
-  const message = 'Unescaped HTML literal. Use html`` tag template literal for secure escaping.'
+module.exports = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'disallow unesaped HTML literals',
+      url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/unescaped-html-literal.md'
+    },
+    schema: []
+  },
 
-  return {
-    Literal(node) {
-      if (!htmlOpenTag.test(node.value)) return
+  create(context) {
+    const htmlOpenTag = /^<[a-zA-Z]/
+    const message = 'Unescaped HTML literal. Use html`` tag template literal for secure escaping.'
 
-      context.report({
-        node,
-        message
-      })
-    },
-    TemplateLiteral(node) {
-      if (!htmlOpenTag.test(node.quasis[0].value.raw)) return
+    return {
+      Literal(node) {
+        if (!htmlOpenTag.test(node.value)) return
 
-      if (!node.parent.tag || node.parent.tag.name !== 'html') {
         context.report({
           node,
           message
         })
+      },
+      TemplateLiteral(node) {
+        if (!htmlOpenTag.test(node.quasis[0].value.raw)) return
+
+        if (!node.parent.tag || node.parent.tag.name !== 'html') {
+          context.report({
+            node,
+            message
+          })
+        }
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -1,11 +1,17 @@`
`1`	`1`	`module.exports = function(context) {`
`2`	`2`	`return {`
	`3`	`+ meta: {`
	`4`	`+ type: 'problem',`
	`5`	`+ docs: {`
	`6`	+ description: 'disallow usage of `Element.prototype.blur()`',
	`7`	`+ url: 'https://github.com/github/eslint-plugin-github/blob/main/docs/rules/no-blur.md'`
	`8`	`+ },`
	`9`	`+ schema: []`
	`10`	`+ },`
`3`	`11`	`CallExpression(node) {`
`4`	`12`	`if (node.callee.property && node.callee.property.name === 'blur') {`
`5`	`13`	`context.report(node, 'Do not use element.blur(), instead restore the focus of a previous element.')`
`6`	`14`	`}`
`7`	`15`	`}`
`8`	`16`	`}`
`9`	`17`	`}`
`10`		`-`
`11`		`-module.exports.schema = []`