Remove generator, fix description

jsx-eslint · Dec 20, 2023 · f3d75e6 · f3d75e6
1 parent d37cfb8
commit f3d75e6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/lib/rules/forward-ref-uses-ref.js b/lib/rules/forward-ref-uses-ref.js
@@ -68,14 +68,14 @@ module.exports = {
               Object.assign(
                 getMessageData('addRefParameter', messages.addRefParameter),
                 {
-                  * fix(fixer) {
+                  fix(fixer) {
                     const param = node.params[0];
                     // If using shorthand arrow function syntax, add parentheses around the new parameter pair
                     const shouldAddParentheses = node.type === 'ArrowFunctionExpression' && !isParenthesized(context, param);
-                    if (shouldAddParentheses) {
-                      yield fixer.insertTextBefore(param, '(');
-                    }
-                    yield fixer.insertTextAfter(param, `, ref${shouldAddParentheses ? ')' : ''}`);
+                    return [].concat(
+                      shouldAddParentheses ? fixer.insertTextBefore(param, '(') : [],
+                      fixer.insertTextAfter(param, `, ref${shouldAddParentheses ? ')' : ''}`),
+                    };
                   },
                 }
               ),

diff --git a/tests/lib/rules/forward-ref-uses-ref.js b/tests/lib/rules/forward-ref-uses-ref.js
@@ -1,6 +1,6 @@
 /**
- * @fileoverview Ensure symmetric naming of setState hook value and setter variables
- * @author Duncan Beevers
+ * @fileoverview Require all forwardRef components include a ref parameter
+ * @author Tiger Oakes
  */
 
 'use strict';