[Refactor] clean up some of the jsdocs

lib/rules/boolean-prop-naming.js

@@ -128,7 +128,7 @@ module.exports = {
     /**
      * Checks if prop is declared in flow way
      * @param {Object} prop Property object, single prop type declaration
-     * @returns {Boolean}
+     * @returns {boolean}
      */
     function flowCheck(prop) {
       return (
@@ -141,7 +141,7 @@ module.exports = {
     /**
      * Checks if prop is declared in regular way
      * @param {Object} prop Property object, single prop type declaration
-     * @returns {Boolean}
+     * @returns {boolean}
      */
     function regularCheck(prop) {
       const propKey = getPropKey(prop);
@@ -165,7 +165,7 @@ module.exports = {
     /**
      * Checks if prop is nested
      * @param {Object} prop Property object, single prop type declaration
-     * @returns {Boolean}
+     * @returns {boolean}
      */
     function nestedPropTypes(prop) {
       return (

lib/rules/display-name.js

@@ -73,7 +73,7 @@ module.exports = {
     /**
      * Checks if React.forwardRef is nested inside React.memo
      * @param {ASTNode} node The AST node being checked.
-     * @returns {Boolean} True if React.forwardRef is nested inside React.memo, false if not.
+     * @returns {boolean} True if React.forwardRef is nested inside React.memo, false if not.
      */
     function isNestedMemo(node) {
       const argumentIsCallExpression = node.arguments && node.arguments[0] && node.arguments[0].type === 'CallExpression';
@@ -111,7 +111,7 @@ module.exports = {
     /**
      * Checks if the component have a name set by the transpiler
      * @param {ASTNode} node The AST node being checked.
-     * @returns {Boolean} True if component has a name, false if not.
+     * @returns {boolean} True if component has a name, false if not.
      */
     function hasTranspilerName(node) {
       const namedObjectAssignment = (

lib/rules/jsx-closing-bracket-location.js

@@ -100,7 +100,7 @@ module.exports = {
     /**
      * Get expected location for the closing bracket
      * @param {Object} tokens Locations of the opening bracket, closing bracket and last prop
-     * @return {String} Expected location for the closing bracket
+     * @return {string} Expected location for the closing bracket
      */
     function getExpectedLocation(tokens) {
       let location;
@@ -121,7 +121,7 @@ module.exports = {
      * Get the correct 0-indexed column for the closing bracket, given the
      * expected location.
      * @param {Object} tokens Locations of the opening bracket, closing bracket and last prop
-     * @param {String} expectedLocation Expected location for the closing bracket
+     * @param {string} expectedLocation Expected location for the closing bracket
      * @return {?Number} The correct column for the closing bracket, or null
      */
     function getCorrectColumn(tokens, expectedLocation) {
@@ -140,8 +140,8 @@ module.exports = {
     /**
      * Check if the closing bracket is correctly located
      * @param {Object} tokens Locations of the opening bracket, closing bracket and last prop
-     * @param {String} expectedLocation Expected location for the closing bracket
-     * @return {Boolean} True if the closing bracket is correctly located, false if not
+     * @param {string} expectedLocation Expected location for the closing bracket
+     * @return {boolean} True if the closing bracket is correctly located, false if not
      */
     function hasCorrectLocation(tokens, expectedLocation) {
       switch (expectedLocation) {
@@ -163,9 +163,9 @@ module.exports = {
     /**
      * Get the characters used for indentation on the line to be matched
      * @param {Object} tokens Locations of the opening bracket, closing bracket and last prop
-     * @param {String} expectedLocation Expected location for the closing bracket
-     * @param {Number} [correctColumn] Expected column for the closing bracket. Default to 0
-     * @return {String} The characters used for indentation
+     * @param {string} expectedLocation Expected location for the closing bracket
+     * @param {number} [correctColumn] Expected column for the closing bracket. Default to 0
+     * @return {string} The characters used for indentation
      */
     function getIndentation(tokens, expectedLocation, correctColumn) {
       const newColumn = correctColumn || 0;
@@ -235,7 +235,7 @@ module.exports = {
      * Get an unique ID for a given JSXOpeningElement
      *
      * @param {ASTNode} node The AST node being checked.
-     * @returns {String} Unique ID (based on its range)
+     * @returns {string} Unique ID (based on its range)
      */
     function getOpeningElementId(node) {
       return node.range.join(':');

lib/rules/jsx-indent-props.js

@@ -116,8 +116,8 @@ module.exports = {
     /**
      * Reports a given indent violation and properly pluralizes the message
      * @param {ASTNode} node Node violating the indent rule
-     * @param {Number} needed Expected indentation character count
-     * @param {Number} gotten Indentation character count in the actual node/code
+     * @param {number} needed Expected indentation character count
+     * @param {number} gotten Indentation character count in the actual node/code
      */
     function report(node, needed, gotten) {
       const msgContext = {
@@ -141,7 +141,7 @@ module.exports = {
     /**
      * Get node indent
      * @param {ASTNode} node Node to examine
-     * @return {Number} Indent
+     * @return {number} Indent
      */
     function getNodeIndent(node) {
       let src = getText(context, node, node.loc.start.column + extraColumnStart);
@@ -173,7 +173,7 @@ module.exports = {
     /**
      * Check indent for nodes list
      * @param {ASTNode[]} nodes list of node objects
-     * @param {Number} indent needed indent
+     * @param {number} indent needed indent
      */
     function checkNodesIndent(nodes, indent) {
       let nestedIndent = indent;

lib/rules/jsx-indent.js

@@ -105,7 +105,7 @@ module.exports = {
     /**
      * Responsible for fixing the indentation issue fix
      * @param {ASTNode} node Node violating the indent rule
-     * @param {Number} needed Expected indentation character count
+     * @param {number} needed Expected indentation character count
      * @returns {Function} function to be executed by the fixer
      * @private
      */
@@ -146,8 +146,8 @@ module.exports = {
     /**
      * Reports a given indent violation and properly pluralizes the message
      * @param {ASTNode} node Node violating the indent rule
-     * @param {Number} needed Expected indentation character count
-     * @param {Number} gotten Indentation character count in the actual node/code
+     * @param {number} needed Expected indentation character count
+     * @param {number} gotten Indentation character count in the actual node/code
      * @param {Object} [loc] Error line and column location
      */
     function report(node, needed, gotten, loc) {
@@ -168,9 +168,9 @@ module.exports = {
     /**
      * Get node indent
      * @param {ASTNode} node Node to examine
-     * @param {Boolean} [byLastLine] get indent of node's last line
-     * @param {Boolean} [excludeCommas] skip comma on start of line
-     * @return {Number} Indent
+     * @param {boolean} [byLastLine] get indent of node's last line
+     * @param {boolean} [excludeCommas] skip comma on start of line
+     * @return {number} Indent
      */
     function getNodeIndent(node, byLastLine, excludeCommas) {
       let src = getText(context, node, node.loc.start.column + extraColumnStart);
@@ -197,7 +197,7 @@ module.exports = {
     /**
      * Check if the node is the right member of a logical expression
      * @param {ASTNode} node The node to check
-     * @return {Boolean} true if its the case, false if not
+     * @return {boolean} true if its the case, false if not
      */
     function isRightInLogicalExp(node) {
       return (
@@ -212,7 +212,7 @@ module.exports = {
     /**
      * Check if the node is the alternate member of a conditional expression
      * @param {ASTNode} node The node to check
-     * @return {Boolean} true if its the case, false if not
+     * @return {boolean} true if its the case, false if not
      */
     function isAlternateInConditionalExp(node) {
       return (
@@ -227,7 +227,7 @@ module.exports = {
     /**
      * Check if the node is within a DoExpression block but not the first expression (which need to be indented)
      * @param {ASTNode} node The node to check
-     * @return {Boolean} true if its the case, false if not
+     * @return {boolean} true if its the case, false if not
      */
     function isSecondOrSubsequentExpWithinDoExp(node) {
       /*
@@ -298,8 +298,8 @@ module.exports = {
     /**
      * Check indent for nodes list
      * @param {ASTNode} node The node to check
-     * @param {Number} indent needed indent
-     * @param {Boolean} [excludeCommas] skip comma on start of line
+     * @param {number} indent needed indent
+     * @param {boolean} [excludeCommas] skip comma on start of line
      */
     function checkNodesIndent(node, indent, excludeCommas) {
       const nodeIndent = getNodeIndent(node, false, excludeCommas);
@@ -318,7 +318,7 @@ module.exports = {
     /**
      * Check indent for Literal Node or JSXText Node
      * @param {ASTNode} node The node to check
-     * @param {Number} indent needed indent
+     * @param {number} indent needed indent
      */
     function checkLiteralNodeIndent(node, indent) {
       const value = node.value;

lib/rules/jsx-no-target-blank.js

@@ -70,7 +70,7 @@ function hasDynamicLink(node, linkAttributes) {
  * Get the string(s) from a value
  * @param {ASTNode} value The AST node being checked.
  * @param {ASTNode} targetValue The AST node being checked.
- * @returns {String | String[] | null} The string value, or null if not a string.
+ * @returns {string | string[] | null} The string value, or null if not a string.
  */
 function getStringFromValue(value, targetValue) {
   if (value) {

lib/rules/jsx-sort-default-props.js

@@ -57,7 +57,7 @@ module.exports = {
     /**
      * Get properties name
      * @param {Object} node - Property.
-     * @returns {String} Property name.
+     * @returns {string} Property name.
      */
     function getPropertyName(node) {
       if (node.key || ['MethodDefinition', 'Property'].indexOf(node.type) !== -1) {
@@ -78,7 +78,7 @@ module.exports = {
     /**
      * Checks if the Identifier node passed in looks like a defaultProps declaration.
      * @param   {ASTNode}  node The node to check. Must be an Identifier node.
-     * @returns {Boolean}       `true` if the node is a defaultProps declaration, `false` if not
+     * @returns {boolean}       `true` if the node is a defaultProps declaration, `false` if not
      */
     function isDefaultPropsDeclaration(node) {
       const propName = getPropertyName(node);

lib/rules/jsx-sort-props.js

@@ -276,8 +276,8 @@ function generateFixerFunction(node, context, reservedList) {
 /**
  * Checks if the `reservedFirst` option is valid
  * @param {Object} context The context of the rule
- * @param {Boolean|Array<String>} reservedFirst The `reservedFirst` option
- * @return {Function|undefined} If an error is detected, a function to generate the error message, otherwise, `undefined`
+ * @param {boolean | string[]} reservedFirst The `reservedFirst` option
+ * @return {Function | undefined} If an error is detected, a function to generate the error message, otherwise, `undefined`
  */
 // eslint-disable-next-line consistent-return
 function validateReservedFirstConfig(context, reservedFirst) {

lib/rules/no-children-prop.js

@@ -17,7 +17,7 @@ const report = require('../util/report');
  * Checks if the node is a createElement call with a props literal.
  * @param {ASTNode} node - The AST node being checked.
  * @param {Context} context - The AST node being checked.
- * @returns {Boolean} - True if node is a createElement call with a props
+ * @returns {boolean} - True if node is a createElement call with a props
  * object literal, False if not.
 */
 function isCreateElementWithProps(node, context) {

lib/rules/no-danger-with-children.js

@@ -85,7 +85,7 @@ module.exports = {
     /**
      * Checks to see if a node is a line break
      * @param {ASTNode} node The AST node being checked
-     * @returns {Boolean} True if node is a line break, false if not
+     * @returns {boolean} True if node is a line break, false if not
      */
     function isLineBreak(node) {
       const isLiteral = node.type === 'Literal' || node.type === 'JSXText';

lib/rules/no-danger.js

@@ -29,7 +29,7 @@ const DANGEROUS_PROPERTIES = fromEntries(DANGEROUS_PROPERTY_NAMES.map((prop) =>
 
 /**
  * Checks if a JSX attribute is dangerous.
- * @param {String} name - Name of the attribute to check.
+ * @param {string} name - Name of the attribute to check.
  * @returns {boolean} Whether or not the attribute is dangerous.
  */
 function isDangerous(name) {

lib/rules/no-direct-mutation-state.js

@@ -38,7 +38,7 @@ module.exports = {
     /**
      * Checks if the component is valid
      * @param {Object} component The component to process
-     * @returns {Boolean} True if the component is valid, false if not.
+     * @returns {boolean} True if the component is valid, false if not.
      */
     function isValid(component) {
       return !!component && !component.mutateSetState;
@@ -73,7 +73,7 @@ module.exports = {
     /**
      * Determine if we should currently ignore assignments in this component.
      * @param {?Object} component The component to process
-     * @returns {Boolean} True if we should skip assignment checks.
+     * @returns {boolean} True if we should skip assignment checks.
      */
     function shouldIgnoreComponent(component) {
       return !component || (component.inConstructor && !component.inCallExpression);

lib/rules/no-multi-comp.js

@@ -50,7 +50,7 @@ module.exports = {
     /**
      * Checks if the component is ignored
      * @param {Object} component The component being checked.
-     * @returns {Boolean} True if the component is ignored, false if not.
+     * @returns {boolean} True if the component is ignored, false if not.
      */
     function isIgnored(component) {
       return (

lib/rules/no-redundant-should-component-update.js

@@ -36,7 +36,7 @@ module.exports = {
     /**
      * Checks for shouldComponentUpdate property
      * @param {ASTNode} node The AST node being checked.
-     * @returns {Boolean} Whether or not the property exists.
+     * @returns {boolean} Whether or not the property exists.
      */
     function hasShouldComponentUpdate(node) {
       const properties = astUtil.getComponentProperties(node);
@@ -49,7 +49,7 @@ module.exports = {
     /**
      * Get name of node if available
      * @param {ASTNode} node The AST node being checked.
-     * @return {String} The name of the node
+     * @return {string} The name of the node
      */
     function getNodeName(node) {
       if (node.id) {

lib/rules/no-set-state.js

@@ -38,7 +38,7 @@ module.exports = {
     /**
      * Checks if the component is valid
      * @param {Object} component The component to process
-     * @returns {Boolean} True if the component is valid, false if not.
+     * @returns {boolean} True if the component is valid, false if not.
      */
     function isValid(component) {
       return !!component && !component.useSetState;

lib/rules/no-string-refs.js

@@ -46,7 +46,7 @@ module.exports = {
     /**
      * Checks if we are using refs
      * @param {ASTNode} node The AST node being checked.
-     * @returns {Boolean} True if we are using refs, false if not.
+     * @returns {boolean} True if we are using refs, false if not.
      */
     function isRefsUsage(node) {
       return !!(
@@ -59,7 +59,7 @@ module.exports = {
     /**
      * Checks if we are using a ref attribute
      * @param {ASTNode} node The AST node being checked.
-     * @returns {Boolean} True if we are using a ref attribute, false if not.
+     * @returns {boolean} True if we are using a ref attribute, false if not.
      */
     function isRefAttribute(node) {
       return !!(
@@ -72,7 +72,7 @@ module.exports = {
     /**
      * Checks if a node contains a string value
      * @param {ASTNode} node The AST node being checked.
-     * @returns {Boolean} True if the node contains a string value, false if not.
+     * @returns {boolean} True if the node contains a string value, false if not.
      */
     function containsStringLiteral(node) {
       return !!(
@@ -85,7 +85,7 @@ module.exports = {
     /**
      * Checks if a node contains a string value within a jsx expression
      * @param {ASTNode} node The AST node being checked.
-     * @returns {Boolean} True if the node contains a string value within a jsx expression, false if not.
+     * @returns {boolean} True if the node contains a string value within a jsx expression, false if not.
      */
     function containsStringExpressionContainer(node) {
       return !!(