feat: drop Node 4, 6, 8, 10, 12, 14 & 16 support

BREAKING CHANGE: Requires Node@^18.18.0 || ^20.9.0 || >=21.1.0

Author: MichaelDeBoey

.eslintrc

@@ -7,7 +7,7 @@
 		"node": true
 	},
 	"parserOptions": {
-		"ecmaVersion": 6,
+		"ecmaVersion": 2022,
 		"ecmaFeatures": {
 			"jsx": true
 		},
@@ -34,17 +34,10 @@
     "consistent-return": 0,
 
     "prefer-destructuring": [2, { "array": false, "object": false }, { "enforceForRenamedProperties": false }],
-    "prefer-object-spread": 0, // until node 8 is required
-    "prefer-rest-params": 0, // until node 6 is required
-    "prefer-spread": 0, // until node 6 is required
-    "function-call-argument-newline": 1, // TODO: enable
+    "function-call-argument-newline": 1,
     "function-paren-newline": 0,
     "no-plusplus": [2, {"allowForLoopAfterthoughts": true}],
     "no-param-reassign": 1,
-    "no-restricted-syntax": [2, {
-      "selector": "ObjectPattern",
-      "message": "Object destructuring is not compatible with Node v4"
-    }],
     "strict": [2, "safe"],
     "valid-jsdoc": [2, {
       "requireReturn": false,
@@ -73,12 +66,11 @@
     {
       "files": ".github/workflows/*.js",
       "parserOptions": {
-        "ecmaVersion": 2019,
+        "ecmaVersion": 2022,
       },
       "rules": {
         "camelcase": 0,
         "no-console": 0,
-        "no-restricted-syntax": 0,
       },
     },
   ],

.github/workflows/node-minors.yml

@@ -1,4 +1,4 @@
-name: 'Tests: node.js (18+)'
+name: 'Tests: node.js'
 
 on: [pull_request, push]
 
@@ -14,7 +14,7 @@ jobs:
         with:
           versionsAsRoot: true
           type: majors
-          preset: '>=18'
+          preset: '^18.18.0 || ^20.9.0 || >=21.1.0'
 
   latest:
     needs: [matrix]

.github/workflows/node-18+.yml renamed to .github/workflows/node.yml

@@ -260,7 +260,7 @@ module.exports = {
       }
 
       const annotationTypeArguments = propsUtil.getTypeArguments(
-        component.node.parent.id.typeAnnotation.typeAnnotation
+        component.node.parent.id.typeAnnotation.typeAnnotation,
       );
       if (
         annotationTypeArguments && (
@@ -269,7 +269,7 @@ module.exports = {
         )
       ) {
         return annotationTypeArguments.params.find(
-          (param) => param.type === 'TSTypeReference' || param.type === 'GenericTypeAnnotation'
+          (param) => param.type === 'TSTypeReference' || param.type === 'GenericTypeAnnotation',
         );
       }
     }
@@ -278,7 +278,7 @@ module.exports = {
       if (node.type === 'TSTypeLiteral' || node.type === 'ObjectTypeAnnotation' || node.type === 'TSInterfaceBody') {
         const currentNode = [].concat(
           objectTypeAnnotations.get(identifier.name) || [],
-          node
+          node,
         );
         objectTypeAnnotations.set(identifier.name, currentNode);
       } else if (
@@ -317,7 +317,7 @@ module.exports = {
           && astUtil.isCallExpression(node.value)
           && propWrapperUtil.isPropWrapperFunction(
             context,
-            getText(context, node.value.callee)
+            getText(context, node.value.callee),
           )
         ) {
           checkPropWrapperArguments(node, node.value.arguments);
@@ -343,7 +343,7 @@ module.exports = {
           astUtil.isCallExpression(right)
           && propWrapperUtil.isPropWrapperFunction(
             context,
-            getText(context, right.callee)
+            getText(context, right.callee),
           )
         ) {
           checkPropWrapperArguments(component.node, right.arguments);
@@ -407,7 +407,7 @@ module.exports = {
               [].concat(propType).filter(Boolean).forEach((prop) => {
                 validatePropNaming(
                   component.node,
-                  prop.properties || prop.members || prop.body
+                  prop.properties || prop.members || prop.body,
                 );
               });
             }

lib/rules/boolean-prop-naming.js

@@ -61,7 +61,7 @@ module.exports = {
   },
 
   create(context) {
-    const configuration = Object.assign({}, optionDefaults, context.options[0]);
+    const configuration = { ...optionDefaults, ...context.options[0] };
 
     function reportMissing(node) {
       report(context, messages.missingType, 'missingType', {

lib/rules/button-has-type.js

@@ -36,8 +36,8 @@ function extractTargetProps(properties, keyName) {
         prop[keyName] && targetPropSet.has(prop[keyName].name)
           ? [prop[keyName].name]
           : []
-      )
-    )
+      ),
+    ),
   );
 }
 
@@ -64,14 +64,14 @@ module.exports = {
     }],
   },
   create(context) {
-    const options = Object.assign({}, defaultOptions, context.options[0]);
+    const options = { ...defaultOptions, ...context.options[0] };
 
     function reportMissingProperty(node) {
       report(
         context,
         messages.missingProperty,
         'missingProperty',
-        { node }
+        { node },
       );
     }
 
@@ -80,7 +80,7 @@ module.exports = {
         context,
         messages.exclusiveCheckedAttribute,
         'exclusiveCheckedAttribute',
-        { node }
+        { node },
       );
     }
 

lib/rules/checked-requires-onchange-or-readonly.js

@@ -100,7 +100,7 @@ module.exports = {
           .forEach((component) => {
             reportInvalidDefaultProps(
               component.declaredPropTypes,
-              component.defaultProps || {}
+              component.defaultProps || {},
             );
           });
       },

lib/rules/default-props-match-prop-types.js

@@ -276,7 +276,7 @@ module.exports = {
           // Report missing display name for all context objects
           forEach(
             filter(contextObjects.values(), (v) => !v.hasDisplayName),
-            (contextObj) => reportMissingContextDisplayName(contextObj)
+            (contextObj) => reportMissingContextDisplayName(contextObj),
           );
         }
       },

lib/rules/display-name.js

@@ -179,7 +179,7 @@ module.exports = {
         }
 
         return options.allowPatternList.every(
-          (pattern) => !minimatch(tagName, pattern)
+          (pattern) => !minimatch(tagName, pattern),
         );
       }
 
@@ -193,7 +193,7 @@ module.exports = {
         }
 
         return options.disallowPatternList.some(
-          (pattern) => minimatch(tagName, pattern)
+          (pattern) => minimatch(tagName, pattern),
         );
       }
 

lib/rules/forbid-component-props.js

@@ -86,7 +86,7 @@ module.exports = {
               element,
               message,
             },
-          }
+          },
         );
       }
     }