@will-stone/eslint-config 5.1.0 → 6.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@will-stone/eslint-config",
-  "version": "5.1.0",
+  "version": "6.2.0",
   "private": false,
   "description": "My ESLint config",
   "repository": {
@@ -9,7 +9,7 @@
   "license": "MIT",
   "author": "Will Stone",
   "scripts": {
-    "eslint-find-unused-rules": "eslint-find-rules --unused ./_for_rule_checking.js > ./unused-rules.txt",
+    "eslint-find-unused-rules": "npx eslint-find-rules --unused ./_for_rule_checking.js > ./unused-rules.txt",
     "eslint-print-rules": "eslint --no-eslintrc -c ./_for_rule_checking.js --print-config ./_for_rule_checking.js > rules.json"
   },
   "dependencies": {
@@ -17,22 +17,22 @@
     "eslint-config-prettier": "^8.3.0"
   },
   "devDependencies": {
-    "eslint-find-rules": "^3.6.1"
+    "eslint-find-rules": "^4.0.0"
   },
   "peerDependencies": {
-    "@typescript-eslint/eslint-plugin": "^5.3.0",
-    "@typescript-eslint/parser": "^5.3.0",
-    "eslint": "^7.32.0",
-    "eslint-plugin-import": "^2.25.2",
-    "eslint-plugin-jest": "^25.2.2",
-    "eslint-plugin-jsx-a11y": "^6.4.1",
+    "@typescript-eslint/eslint-plugin": "^5.4.0",
+    "@typescript-eslint/parser": "^5.4.0",
+    "eslint": "^8.3.0",
+    "eslint-plugin-import": "^2.25.3",
+    "eslint-plugin-jest": "^25.2.4",
+    "eslint-plugin-jsx-a11y": "^6.5.1",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^4.0.0",
-    "eslint-plugin-react": "^7.26.1",
-    "eslint-plugin-react-hooks": "^4.2.0",
+    "eslint-plugin-react": "^7.27.1",
+    "eslint-plugin-react-hooks": "^4.3.0",
     "eslint-plugin-simple-import-sort": "^7.0.0",
     "eslint-plugin-switch-case": "^1.1.2",
-    "eslint-plugin-unicorn": "^38.0.0",
+    "eslint-plugin-unicorn": "^39.0.0",
     "typescript": "4.x"
   },
   "peerDependenciesMeta": {

package/rules/built-in.js

@@ -584,6 +584,8 @@ module.exports = {
   // likely an error due to incomplete refactoring.
   'no-unused-labels': 'warn',
 
+  'no-unused-private-class-members': 'error',
+
   // Variables must be used unless name starts with "ignored"
   'no-unused-vars': ['error', { varsIgnorePattern: '[iI]gnored' }],
 

package/rules/react.js

@@ -49,12 +49,8 @@ module.exports = {
   'react/react-in-jsx-scope': 'off',
 
   // Enforce a defaultProps definition for every prop that is not a required prop
-  'react/require-default-props': [
-    'error',
-    {
-      ignoreFunctionalComponents: true,
-    },
-  ],
+  // TODO turn this back on and create a react-typescript rule set for disabling thing like this.
+  'react/require-default-props': 'off',
 
   // Do not enforce React components to have a shouldComponentUpdate method.
   'react/require-optimization': 'off',

package/rules/typescript.js

@@ -77,6 +77,7 @@ module.exports = {
    * These are for type checking which I don't use eslint for
    */
   '@typescript-eslint/await-thenable': 'off',
+  '@typescript-eslint/consistent-type-exports': 'off',
   '@typescript-eslint/dot-notation': 'off',
   '@typescript-eslint/naming-convention': 'off',
   '@typescript-eslint/no-base-to-string': 'off',

package/rules/unicorn.js

@@ -67,6 +67,9 @@ module.exports = {
   // for..of is much easier to read
   'unicorn/no-array-reduce': 'error',
 
+  // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-await-expression-member.md
+  'unicorn/no-await-expression-member': 'warn',
+
   // Do not use leading/trailing space between console.log parameters
   'unicorn/no-console-spaces': 'warn',
 
@@ -177,6 +180,10 @@ module.exports = {
   // Prefer .at() method for index access and String#charAt()
   'unicorn/prefer-at': 'warn',
 
+  // Prefer String#codePointAt(…) over String#charCodeAt(…)
+  // and String.fromCodePoint(…) over String.fromCharCode(…)
+  'unicorn/prefer-code-point': 'error',
+
   // Prefer Date.now() to get the number of milliseconds since the Unix Epoch
   'unicorn/prefer-date-now': 'warn',
 
@@ -285,6 +292,8 @@ module.exports = {
     {
       replacements: {
         props: false,
+        ref: false,
+        refs: false,
       },
       allowList: {
         'next-env.d': true,