[eslint] change comma-dangle to always-multiline

Author: ljharb

.eslintrc

@@ -14,7 +14,7 @@
       "sourceType": "script",
     },
     "rules": {
-        "comma-dangle": [2, "never"],
+        "comma-dangle": [2, "always-multiline"],
         "object-curly-spacing": [2, "never"],
         "object-shorthand": [2, "always"],
         "array-bracket-spacing": [2, "never"],

index.js

@@ -98,7 +98,7 @@ const allRules = {
   'state-in-constructor': require('./lib/rules/state-in-constructor'),
   'static-property-placement': require('./lib/rules/static-property-placement'),
   'style-prop-object': require('./lib/rules/style-prop-object'),
-  'void-dom-elements-no-children': require('./lib/rules/void-dom-elements-no-children')
+  'void-dom-elements-no-children': require('./lib/rules/void-dom-elements-no-children'),
 };
 /* eslint-enable */
 
@@ -121,12 +121,12 @@ module.exports = {
   configs: {
     recommended: {
       plugins: [
-        'react'
+        'react',
       ],
       parserOptions: {
         ecmaFeatures: {
-          jsx: true
-        }
+          jsx: true,
+        },
       },
       rules: {
         'react/display-name': 2,
@@ -150,34 +150,34 @@ module.exports = {
         'react/no-unsafe': 0,
         'react/prop-types': 2,
         'react/react-in-jsx-scope': 2,
-        'react/require-render-return': 2
-      }
+        'react/require-render-return': 2,
+      },
     },
     all: {
       plugins: [
-        'react'
+        'react',
       ],
       parserOptions: {
         ecmaFeatures: {
-          jsx: true
-        }
+          jsx: true,
+        },
       },
-      rules: activeRulesConfig
+      rules: activeRulesConfig,
     },
     'jsx-runtime': {
       plugins: [
-        'react'
+        'react',
       ],
       parserOptions: {
         ecmaFeatures: {
-          jsx: true
+          jsx: true,
         },
-        jsxPragma: null // for @typescript/eslint-parser
+        jsxPragma: null, // for @typescript/eslint-parser
       },
       rules: {
         'react/react-in-jsx-scope': 0,
-        'react/jsx-uses-react': 0
-      }
-    }
-  }
+        'react/jsx-uses-react': 0,
+      },
+    },
+  },
 };

lib/rules/boolean-prop-naming.js

@@ -16,7 +16,7 @@ const report = require('../util/report');
 // ------------------------------------------------------------------------------
 
 const messages = {
-  patternMismatch: 'Prop name ({{propName}}) doesn\'t match rule ({{pattern}})'
+  patternMismatch: 'Prop name ({{propName}}) doesn\'t match rule ({{pattern}})',
 };
 
 module.exports = {
@@ -25,7 +25,7 @@ module.exports = {
       category: 'Stylistic Issues',
       description: 'Enforces consistent naming for boolean props',
       recommended: false,
-      url: docsUrl('boolean-prop-naming')
+      url: docsUrl('boolean-prop-naming'),
     },
 
     messages,
@@ -35,28 +35,28 @@ module.exports = {
       properties: {
         propTypeNames: {
           items: {
-            type: 'string'
+            type: 'string',
           },
           minItems: 1,
           type: 'array',
-          uniqueItems: true
+          uniqueItems: true,
         },
         rule: {
           default: '^(is|has)[A-Z]([A-Za-z0-9]?)+',
           minLength: 1,
-          type: 'string'
+          type: 'string',
         },
         message: {
           minLength: 1,
-          type: 'string'
+          type: 'string',
         },
         validateNested: {
           default: false,
-          type: 'boolean'
-        }
+          type: 'boolean',
+        },
       },
-      type: 'object'
-    }]
+      type: 'object',
+    }],
   },
 
   create: Components.detect((context, components, utils) => {
@@ -199,7 +199,7 @@ module.exports = {
       });
 
       components.set(node, {
-        invalidProps
+        invalidProps,
       });
     }
 
@@ -215,8 +215,8 @@ module.exports = {
           data: {
             component: propName,
             propName,
-            pattern: config.rule
-          }
+            pattern: config.rule,
+          },
         });
       });
     }
@@ -348,7 +348,7 @@ module.exports = {
 
         // Reset cache
         objectTypeAnnotations.clear();
-      }
+      },
     };
-  })
+  }),
 };

lib/rules/button-has-type.js

@@ -18,14 +18,14 @@ const report = require('../util/report');
 const optionDefaults = {
   button: true,
   submit: true,
-  reset: true
+  reset: true,
 };
 
 const messages = {
   missingType: 'Missing an explicit type attribute for button',
   complexType: 'The button type attribute must be specified by a static string or a trivial ternary expression',
   invalidValue: '"{{value}}" is an invalid value for button type attribute',
-  forbiddenValue: '"{{value}}" is an invalid value for button type attribute'
+  forbiddenValue: '"{{value}}" is an invalid value for button type attribute',
 };
 
 module.exports = {
@@ -34,7 +34,7 @@ module.exports = {
       description: 'Forbid "button" element without an explicit "type" attribute',
       category: 'Possible Errors',
       recommended: false,
-      url: docsUrl('button-has-type')
+      url: docsUrl('button-has-type'),
     },
 
     messages,
@@ -44,33 +44,33 @@ module.exports = {
       properties: {
         button: {
           default: optionDefaults.button,
-          type: 'boolean'
+          type: 'boolean',
         },
         submit: {
           default: optionDefaults.submit,
-          type: 'boolean'
+          type: 'boolean',
         },
         reset: {
           default: optionDefaults.reset,
-          type: 'boolean'
-        }
+          type: 'boolean',
+        },
       },
-      additionalProperties: false
-    }]
+      additionalProperties: false,
+    }],
   },
 
   create(context) {
     const configuration = Object.assign({}, optionDefaults, context.options[0]);
 
     function reportMissing(node) {
       report(context, messages.missingType, 'missingType', {
-        node
+        node,
       });
     }
 
     function reportComplex(node) {
       report(context, messages.complexType, 'complexType', {
-        node
+        node,
       });
     }
 
@@ -79,15 +79,15 @@ module.exports = {
         report(context, messages.invalidValue, 'invalidValue', {
           node,
           data: {
-            value
-          }
+            value,
+          },
         });
       } else if (!configuration[value]) {
         report(context, messages.forbiddenValue, 'forbiddenValue', {
           node,
           data: {
-            value
-          }
+            value,
+          },
         });
       }
     }
@@ -157,7 +157,7 @@ module.exports = {
         }
 
         checkExpression(node, typeProp.value);
-      }
+      },
     };
-  }
+  },
 };

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

@@ -16,15 +16,15 @@ const report = require('../util/report');
 
 const messages = {
   requiredHasDefault: 'defaultProp "{{name}}" defined for isRequired propType.',
-  defaultHasNoType: 'defaultProp "{{name}}" has no corresponding propTypes declaration.'
+  defaultHasNoType: 'defaultProp "{{name}}" has no corresponding propTypes declaration.',
 };
 
 module.exports = {
   meta: {
     docs: {
       description: 'Enforce all defaultProps are defined and not "required" in propTypes.',
       category: 'Best Practices',
-      url: docsUrl('default-props-match-prop-types')
+      url: docsUrl('default-props-match-prop-types'),
     },
 
     messages,
@@ -34,11 +34,11 @@ module.exports = {
       properties: {
         allowRequiredDefaults: {
           default: false,
-          type: 'boolean'
-        }
+          type: 'boolean',
+        },
       },
-      additionalProperties: false
-    }]
+      additionalProperties: false,
+    }],
   },
 
   create: Components.detect((context, components) => {
@@ -71,15 +71,15 @@ module.exports = {
           report(context, messages.requiredHasDefault, 'requiredHasDefault', {
             node: defaultProp.node,
             data: {
-              name: defaultPropName
-            }
+              name: defaultPropName,
+            },
           });
         } else {
           report(context, messages.defaultHasNoType, 'defaultHasNoType', {
             node: defaultProp.node,
             data: {
-              name: defaultPropName
-            }
+              name: defaultPropName,
+            },
           });
         }
       });
@@ -100,7 +100,7 @@ module.exports = {
             list[component].defaultProps || {}
           );
         });
-      }
+      },
     };
-  })
+  }),
 };