@will-stone/eslint-config 5.0.3 → 6.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@will-stone/eslint-config",
-  "version": "5.0.3",
+  "version": "6.0.1",
   "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.2.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": "^38.0.1",
     "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' }],
 
@@ -658,6 +660,7 @@ module.exports = {
   'padding-line-between-statements': [
     'warn',
     { blankLine: 'always', prev: 'multiline-block-like', next: '*' },
+    { blankLine: 'always', prev: '*', next: 'multiline-block-like' },
   ],
 
   // Can cause issues when Prettier is enabled

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

@@ -37,7 +37,7 @@ module.exports = {
   // ts transpiles let/const to var, so no need for vars any more
   'no-var': 'error',
   // ts provides better types with const
-  'prefer-const': 'error',
+  'prefer-const': 'warn',
   // ts provides better types with rest args over arguments
   'prefer-rest-params': 'error',
   // ts transpiles spread to apply, so no need for manual apply
@@ -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',