Merge branch 'main' into smockle/wip

Author: smockle

lib/configs/browser.js

@@ -1,6 +1,6 @@
 module.exports = {
   env: {
-    browser: true
+    browser: true,
   },
   plugins: ['github', 'escompat'],
   extends: ['plugin:escompat/recommended'],
@@ -22,8 +22,8 @@ module.exports = {
       'error',
       {
         selector: "NewExpression[callee.name='URL'][arguments.length=1]",
-        message: 'Please pass in `window.location.origin` as the 2nd argument to `new URL()`'
-      }
-    ]
-  }
+        message: 'Please pass in `window.location.origin` as the 2nd argument to `new URL()`',
+      },
+    ],
+  },
 }

lib/configs/internal.js

@@ -3,6 +3,6 @@ module.exports = {
   rules: {
     'github/authenticity-token': 'error',
     'github/js-class-name': 'error',
-    'github/no-d-none': 'error'
-  }
+    'github/no-d-none': 'error',
+  },
 }

lib/configs/recommended.js

@@ -1,12 +1,12 @@
 module.exports = {
   parserOptions: {
     ecmaFeatures: {
-      ecmaVersion: 6
+      ecmaVersion: 6,
     },
-    sourceType: 'module'
+    sourceType: 'module',
   },
   env: {
-    es6: true
+    es6: true,
   },
   plugins: ['github', 'prettier', 'eslint-comments', 'import', 'filenames', 'i18n-text', 'no-only-tests'],
   rules: {
@@ -41,8 +41,8 @@ module.exports = {
         allowArray: true,
         allowArrowFunction: false,
         allowLiteral: true,
-        allowObject: true
-      }
+        allowObject: true,
+      },
     ],
     'import/no-commonjs': 'error',
     'import/no-deprecated': 'error',
@@ -89,8 +89,8 @@ module.exports = {
     'no-only-tests/no-only-tests': [
       'error',
       {
-        block: ['describe', 'it', 'context', 'test', 'tape', 'fixture', 'serial', 'suite']
-      }
+        block: ['describe', 'it', 'context', 'test', 'tape', 'fixture', 'serial', 'suite'],
+      },
     ],
     'no-redeclare': 'error',
     'no-regex-spaces': 'error',
@@ -122,13 +122,13 @@ module.exports = {
     'use-isnan': 'error',
     'valid-typeof': 'error',
     camelcase: ['error', {properties: 'always'}],
-    eqeqeq: ['error', 'smart']
+    eqeqeq: ['error', 'smart'],
   },
   settings: {
     'import/resolver': {
       node: {
-        extensions: ['.js', '.ts']
-      }
-    }
-  }
+        extensions: ['.js', '.ts'],
+      },
+    },
+  },
 }

lib/configs/typescript.js

@@ -16,6 +16,6 @@ module.exports = {
     '@typescript-eslint/explicit-function-return-type': 'off',
     '@typescript-eslint/no-non-null-assertion': 'off',
     '@typescript-eslint/no-unused-vars': 'error',
-    '@typescript-eslint/explicit-module-boundary-types': 'off'
-  }
+    '@typescript-eslint/explicit-module-boundary-types': 'off',
+  },
 }

lib/formatters/stylish-fixes.js

@@ -65,7 +65,7 @@ module.exports = function (results) {
       ', ',
       warnings,
       pluralize(' warning', warnings),
-      ')\n'
+      ')\n',
     ].join('')
   }
 

lib/index.js

@@ -19,13 +19,13 @@ module.exports = {
     'prefer-observers': require('./rules/prefer-observers'),
     'role-supports-aria-props': require('./rules/role-supports-aria-props'),
     'require-passive-events': require('./rules/require-passive-events'),
-    'unescaped-html-literal': require('./rules/unescaped-html-literal')
+    'unescaped-html-literal': require('./rules/unescaped-html-literal'),
   },
   configs: {
     browser: require('./configs/browser'),
     internal: require('./configs/internal'),
     recommended: require('./configs/recommended'),
     typescript: require('./configs/typescript'),
-    react: require('./configs/react')
-  }
+    react: require('./configs/react'),
+  },
 }

lib/rules/a11y-no-generic-link-text.js

@@ -16,9 +16,9 @@ module.exports = {
   meta: {
     docs: {
       description: 'disallow generic link text',
-      url: require('../url')(module)
+      url: require('../url')(module),
     },
-    schema: []
+    schema: [],
   },
 
   create(context) {
@@ -46,13 +46,13 @@ module.exports = {
             context.report({
               node,
               message:
-                'Avoid setting generic link text like `Here`, `Click here`, `Read more`. Make sure that your link text is both descriptive and concise.'
+                'Avoid setting generic link text like `Here`, `Click here`, `Read more`. Make sure that your link text is both descriptive and concise.',
             })
           }
           if (cleanTextContent && !cleanAriaLabelValue.includes(cleanTextContent)) {
             context.report({
               node,
-              message: 'When using ARIA to set a more descriptive text, it must fully contain the visible label.'
+              message: 'When using ARIA to set a more descriptive text, it must fully contain the visible label.',
             })
           }
         } else {
@@ -61,11 +61,11 @@ module.exports = {
             context.report({
               node: jsxTextNode,
               message:
-                'Avoid setting generic link text like `Here`, `Click here`, `Read more`. Make sure that your link text is both descriptive and concise.'
+                'Avoid setting generic link text like `Here`, `Click here`, `Read more`. Make sure that your link text is both descriptive and concise.',
             })
           }
         }
-      }
+      },
     }
-  }
+  },
 }

lib/rules/array-foreach.js

@@ -3,9 +3,9 @@ module.exports = {
     type: 'suggestion',
     docs: {
       description: 'enforce `for..of` loops over `Array.forEach`',
-      url: require('../url')(module)
+      url: require('../url')(module),
     },
-    schema: []
+    schema: [],
   },
 
   create(context) {
@@ -14,7 +14,7 @@ module.exports = {
         if (node.callee.property && node.callee.property.name === 'forEach') {
           context.report({node, message: 'Prefer for...of instead of Array.forEach'})
         }
-      }
+      },
     }
-  }
+  },
 }

lib/rules/async-currenttarget.js

@@ -3,9 +3,9 @@ module.exports = {
     type: 'problem',
     docs: {
       description: 'disallow `event.currentTarget` calls inside of async functions',
-      url: require('../url')(module)
+      url: require('../url')(module),
     },
-    schema: []
+    schema: [],
   },
 
   create(context) {
@@ -22,7 +22,7 @@ module.exports = {
             context.report({node, message: 'event.currentTarget inside an async function is error prone'})
           }
         }
-      }
+      },
     }
-  }
+  },
 }

lib/rules/async-preventdefault.js

@@ -3,9 +3,9 @@ module.exports = {
     type: 'problem',
     docs: {
       description: 'disallow `event.preventDefault` calls inside of async functions',
-      url: require('../url')(module)
+      url: require('../url')(module),
     },
-    schema: []
+    schema: [],
   },
 
   create(context) {
@@ -22,7 +22,7 @@ module.exports = {
             context.report({node, message: 'event.preventDefault() inside an async function is error prone'})
           }
         }
-      }
+      },
     }
-  }
+  },
 }