feat: add classes: ignore option

Author: nix6839

lib/rules/require-default-props.js

@@ -40,13 +40,9 @@ module.exports = {
         forbidDefaultForRequired: {
           type: 'boolean',
         },
-        /**
-         * For now, this wouldn't support any other choices, and would be optional,
-         * but later could be set to "ignore" as long as `functions` wasn't set to "ignore"
-         */
         classes: {
           allow: {
-            enum: ['defaultProps'],
+            enum: ['defaultProps', 'ignore'],
           },
         },
         functions: {
@@ -68,6 +64,7 @@ module.exports = {
   create: Components.detect((context, components) => {
     const configuration = context.options[0] || {};
     const forbidDefaultForRequired = configuration.forbidDefaultForRequired || false;
+    const classes = configuration.classes || 'defaultProps';
     /**
      * @todo
      * - Remove ignoreFunctionalComponents
@@ -177,6 +174,9 @@ module.exports = {
           if (functions === 'ignore' && astUtil.isFunctionLike(component.node)) {
             return false;
           }
+          if (classes === 'ignore' && astUtil.isClass(component.node)) {
+            return false;
+          }
           // If this defaultProps is "unresolved", then we should ignore this component and not report
           // any errors for it, to avoid false-positives with e.g. external defaultProps declarations or spread operators.
           if (component.defaultProps === 'unresolved') {

tests/lib/rules/require-default-props.js

@@ -649,6 +649,56 @@ ruleTester.run('require-default-props', rule, {
       `,
       options: [{ functions: 'defaultArguments' }],
     },
+    {
+      code: `
+        class Greeting extends React.Component {
+          render() {
+            return (
+              <h1>Hello, {this.props.foo} {this.props.bar}</h1>
+            );
+          }
+        }
+        Greeting.propTypes = {
+          foo: PropTypes.string,
+          bar: PropTypes.string.isRequired
+        };
+      `,
+      options: [{ classes: 'ignore' }],
+    },
+    {
+      code: `
+        class Greeting extends React.Component {
+          render() {
+            return (
+              <h1>Hello, {this.props.foo} {this.props.bar}</h1>
+            );
+          }
+        }
+        Greeting.propTypes = {
+          foo: PropTypes.string,
+          bar: PropTypes.string
+        };
+        Greeting.defaultProps = {
+          foo: "foo"
+        };
+      `,
+      options: [{ classes: 'ignore' }],
+    },
+    {
+      code: `
+        class Hello extends React.Component {
+          static get propTypes() {
+            return {
+              name: PropTypes.string
+            };
+          }
+          render() {
+            return <div>Hello {this.props.name}</div>;
+          }
+        }
+      `,
+      options: [{ classes: 'ignore' }],
+    },
 
     // edge cases
 
@@ -676,6 +726,20 @@ ruleTester.run('require-default-props', rule, {
         };
       `,
     },
+    {
+      code: `
+        let Greetings = {};
+        Greetings.Hello = class extends React.Component {
+          render () {
+            return <div>Hello {this.props.foo}</div>;
+          }
+        }
+        Greetings.Hello.propTypes = {
+          foo: PropTypes.string
+        };
+      `,
+      options: [{ classes: 'ignore' }],
+    },
     // external references
     {
       code: `