chore: upgrade a few dev deps to latest (#472)

chore: re-apply prettier
eslint-community · Apr 18, 2024 · 95b1980 · 95b1980
1 parent 189f75c
commit 95b1980
Show file tree

Hide file tree

Showing 37 changed files with 185 additions and 185 deletions.
diff --git a/docs/rules/report-message-format.md b/docs/rules/report-message-format.md
@@ -40,7 +40,7 @@ module.exports = {
     context.report(
       node,
       'This will get reported, regardless of the value of the {{placeholder}}',
-      { placeholder: foo }
+      { placeholder: foo },
     );
   },
 };

diff --git a/eslint.config.js b/eslint.config.js
@@ -17,7 +17,7 @@ module.exports = [
     'not-an-aardvark/node',
     'plugin:eslint-comments/recommended',
     'plugin:prettier/recommended',
-    'plugin:unicorn/recommended'
+    'plugin:unicorn/recommended',
   ),
   pluginN.configs['flat/recommended'],
   {

diff --git a/lib/index.js b/lib/index.js
@@ -39,7 +39,7 @@ const allRules = Object.fromEntries(
     .map((ruleName) => [
       ruleName,
       require(path.join(__dirname, 'rules', ruleName)),
-    ])
+    ]),
 );
 
 const plugin = {
@@ -61,11 +61,11 @@ Object.assign(
         rules: Object.fromEntries(
           Object.keys(allRules)
             .filter((ruleName) => configFilters[configName](allRules[ruleName]))
-            .map((ruleName) => [`${PLUGIN_NAME}/${ruleName}`, 'error'])
+            .map((ruleName) => [`${PLUGIN_NAME}/${ruleName}`, 'error']),
         ),
       },
     });
-  }, {})
+  }, {}),
 );
 
 // flat configs
@@ -79,11 +79,11 @@ Object.assign(
         rules: Object.fromEntries(
           Object.keys(allRules)
             .filter((ruleName) => configFilters[configName](allRules[ruleName]))
-            .map((ruleName) => [`${PLUGIN_NAME}/${ruleName}`, 'error'])
+            .map((ruleName) => [`${PLUGIN_NAME}/${ruleName}`, 'error']),
         ),
       },
     });
-  }, {})
+  }, {}),
 );
 
 module.exports = plugin;
diff --git a/lib/rules/consistent-output.js b/lib/rules/consistent-output.js
@@ -45,11 +45,11 @@ module.exports = {
       Program(ast) {
         utils.getTestInfo(context, ast).forEach((testRun) => {
           const readableCases = testRun.invalid.filter(
-            (testCase) => testCase.type === 'ObjectExpression'
+            (testCase) => testCase.type === 'ObjectExpression',
           );
           const casesWithoutOutput = readableCases.filter(
             (testCase) =>
-              !testCase.properties.map(utils.getKeyName).includes('output')
+              !testCase.properties.map(utils.getKeyName).includes('output'),
           );
 
           if (

diff --git a/lib/rules/fixer-return.js b/lib/rules/fixer-return.js
@@ -54,7 +54,7 @@ module.exports = {
      */
     function ensureFunctionReturnedFix(
       node,
-      loc = (node.id || node).loc.start
+      loc = (node.id || node).loc.start,
     ) {
       if (
         (node.generator && !funcInfo.hasYieldWithFixer) || // Generator function never yielded a fix
@@ -102,7 +102,7 @@ module.exports = {
         const sourceCode = context.sourceCode || context.getSourceCode(); // TODO: just use context.sourceCode when dropping eslint < v9
         contextIdentifiers = utils.getContextIdentifiers(
           sourceCode.scopeManager,
-          ast
+          ast,
         );
       },
 

diff --git a/lib/rules/meta-property-ordering.js b/lib/rules/meta-property-ordering.js
@@ -78,10 +78,10 @@ module.exports = {
         const knownProps = props
           .filter((prop) => orderMap.has(getKeyName(prop)))
           .sort(
-            (a, b) => orderMap.get(getKeyName(a)) - orderMap.get(getKeyName(b))
+            (a, b) => orderMap.get(getKeyName(a)) - orderMap.get(getKeyName(b)),
           );
         const unknownProps = props.filter(
-          (prop) => !orderMap.has(getKeyName(prop))
+          (prop) => !orderMap.has(getKeyName(prop)),
         );
 
         for (const violatingProp of violatingProps) {
@@ -96,7 +96,7 @@ module.exports = {
               return props.map((prop, k) => {
                 return fixer.replaceText(
                   prop,
-                  sourceCode.getText(expectedProps[k])
+                  sourceCode.getText(expectedProps[k]),
                 );
               });
             },

diff --git a/lib/rules/no-deprecated-context-methods.js b/lib/rules/no-deprecated-context-methods.js
@@ -70,8 +70,8 @@ module.exports = {
               contextId.parent.property.type === 'Identifier' &&
               Object.prototype.hasOwnProperty.call(
                 DEPRECATED_PASSTHROUGHS,
-                contextId.parent.property.name
-              )
+                contextId.parent.property.name,
+              ),
           )
           .forEach((contextId) =>
             context.report({
@@ -87,10 +87,10 @@ module.exports = {
                 fixer.insertTextAfter(contextId, '.getSourceCode()'),
                 fixer.replaceText(
                   contextId.parent.property,
-                  DEPRECATED_PASSTHROUGHS[contextId.parent.property.name]
+                  DEPRECATED_PASSTHROUGHS[contextId.parent.property.name],
                 ),
               ],
-            })
+            }),
           );
       },
     };

diff --git a/lib/rules/no-deprecated-report-api.js b/lib/rules/no-deprecated-report-api.js
@@ -41,7 +41,7 @@ module.exports = {
       Program(ast) {
         contextIdentifiers = utils.getContextIdentifiers(
           sourceCode.scopeManager,
-          ast
+          ast,
         );
       },
       CallExpression(node) {
@@ -70,9 +70,9 @@ module.exports = {
                 [openingParen.range[1], closingParen.range[0]],
                 `{${Object.keys(reportInfo)
                   .map(
-                    (key) => `${key}: ${sourceCode.getText(reportInfo[key])}`
+                    (key) => `${key}: ${sourceCode.getText(reportInfo[key])}`,
                   )
-                  .join(', ')}}`
+                  .join(', ')}}`,
               );
             },
           });

diff --git a/lib/rules/no-identical-tests.js b/lib/rules/no-identical-tests.js
@@ -71,7 +71,7 @@ module.exports = {
                       [
                         start.range[1],
                         end.value === ',' ? end.range[1] : test.range[1],
-                      ]
+                      ],
                     );
                   },
                 });

diff --git a/lib/rules/no-missing-message-ids.js b/lib/rules/no-missing-message-ids.js
@@ -64,7 +64,7 @@ module.exports = {
           const reportMessagesAndDataArray =
             utils.collectReportViolationAndSuggestionData(reportInfo);
           for (const { messageId } of reportMessagesAndDataArray.filter(
-            (obj) => obj.messageId
+            (obj) => obj.messageId,
           )) {
             const values =
               messageId.type === 'Literal'
@@ -81,7 +81,7 @@ module.exports = {
                   val.value,
                   ruleInfo,
                   scopeManager,
-                  scope
+                  scope,
                 )
               )
                 // Couldn't find this messageId in `meta.messages`.

diff --git a/lib/rules/no-missing-placeholders.js b/lib/rules/no-missing-placeholders.js
@@ -76,7 +76,7 @@ module.exports = {
                   obj.messageId.value,
                   ruleInfo,
                   scopeManager,
-                  scope
+                  scope,
                 );
                 if (correspondingMessage) {
                   obj.message = correspondingMessage.value;
@@ -105,13 +105,13 @@ module.exports = {
 
               while (
                 (match = PLACEHOLDER_MATCHER.exec(
-                  message.value || messageStaticValue.value
+                  message.value || messageStaticValue.value,
                 ))
               ) {
                 const matchingProperty =
                   data &&
                   data.properties.find(
-                    (prop) => utils.getKeyName(prop) === match[1]
+                    (prop) => utils.getKeyName(prop) === match[1],
                   );
 
                 if (!matchingProperty) {

diff --git a/lib/rules/no-only-tests.js b/lib/rules/no-only-tests.js
@@ -40,7 +40,7 @@ module.exports = {
                   property.key.type === 'Identifier' &&
                   property.key.name === 'only' &&
                   property.value.type === 'Literal' &&
-                  property.value.value
+                  property.value.value,
               );
 
               if (onlyProperty) {

diff --git a/lib/rules/no-property-in-node.js b/lib/rules/no-property-in-node.js
@@ -32,7 +32,7 @@ function isAstNodeType(type) {
   return (type.types || [type])
     .filter((typePart) => typePart.getProperty('type'))
     .flatMap(
-      (typePart) => (typePart.symbol && typePart.symbol.declarations) || []
+      (typePart) => (typePart.symbol && typePart.symbol.declarations) || [],
     )
     .some((declaration) => {
       const fileName = declaration.getSourceFile().fileName;
@@ -69,7 +69,7 @@ module.exports = {
         const services = (context.sourceCode || context).parserServices;
         if (!services.program) {
           throw new Error(
-            'You have used a rule which requires parserServices to be generated. You must therefore provide a value for the "parserOptions.project" property for @typescript-eslint/parser.'
+            'You have used a rule which requires parserServices to be generated. You must therefore provide a value for the "parserOptions.project" property for @typescript-eslint/parser.',
           );
         }
 

diff --git a/lib/rules/no-unused-message-ids.js b/lib/rules/no-unused-message-ids.js
@@ -60,7 +60,7 @@ module.exports = {
         const scope = sourceCode.getScope?.(ast) || context.getScope(); // TODO: just use sourceCode.getScope() when we drop support for ESLint < 9.0.0
 
         const messageIdNodesUnused = messageIdNodes.filter(
-          (node) => !messageIdsUsed.has(utils.getKeyName(node, scope))
+          (node) => !messageIdsUsed.has(utils.getKeyName(node, scope)),
         );
 
         // Report any messageIds that were never used.
@@ -93,7 +93,7 @@ module.exports = {
           const reportMessagesAndDataArray =
             utils.collectReportViolationAndSuggestionData(reportInfo);
           for (const { messageId } of reportMessagesAndDataArray.filter(
-            (obj) => obj.messageId
+            (obj) => obj.messageId,
           )) {
             const values =
               messageId.type === 'Literal'

diff --git a/lib/rules/no-unused-placeholders.js b/lib/rules/no-unused-placeholders.js
@@ -75,7 +75,7 @@ module.exports = {
                   obj.messageId.value,
                   ruleInfo,
                   scopeManager,
-                  scope
+                  scope,
                 );
                 if (correspondingMessage) {
                   obj.message = correspondingMessage.value;
@@ -85,7 +85,7 @@ module.exports = {
           }
 
           for (const { message, data } of reportMessagesAndDataArray.filter(
-            (obj) => obj.message
+            (obj) => obj.message,
           )) {
             const messageStaticValue = getStaticValue(message, scope);
             if (
@@ -105,7 +105,7 @@ module.exports = {
                 PLACEHOLDER_MATCHER,
                 (fullMatch, term) => {
                   placeholdersInMessage.add(term);
-                }
+                },
               );
 
               data.properties.forEach((prop) => {

diff --git a/lib/rules/no-useless-token-range.js b/lib/rules/no-useless-token-range.js
@@ -138,23 +138,23 @@ module.exports = {
               ((isStartAccess(identifier.parent.parent.parent) &&
                 identifier.parent.property.name === 'getFirstToken') ||
                 (isEndAccess(identifier.parent.parent.parent) &&
-                  identifier.parent.property.name === 'getLastToken'))
+                  identifier.parent.property.name === 'getLastToken')),
           )
           .forEach((identifier) => {
             const fullRangeAccess = isRangeAccess(
-              identifier.parent.parent.parent
+              identifier.parent.parent.parent,
             )
               ? identifier.parent.parent.parent.parent
               : identifier.parent.parent.parent;
             const replacementText =
               sourceCode.text.slice(
                 fullRangeAccess.range[0],
-                identifier.parent.parent.range[0]
+                identifier.parent.parent.range[0],
               ) +
               sourceCode.getText(identifier.parent.parent.arguments[0]) +
               sourceCode.text.slice(
                 identifier.parent.parent.range[1],
-                fullRangeAccess.range[1]
+                fullRangeAccess.range[1],
               );
             context.report({
               node: identifier.parent.parent,
@@ -163,7 +163,7 @@ module.exports = {
               fix(fixer) {
                 return fixer.replaceText(
                   identifier.parent.parent,
-                  sourceCode.getText(identifier.parent.parent.arguments[0])
+                  sourceCode.getText(identifier.parent.parent.arguments[0]),
                 );
               },
             });

diff --git a/lib/rules/prefer-message-ids.js b/lib/rules/prefer-message-ids.js
@@ -46,15 +46,15 @@ module.exports = {
         const scope = sourceCode.getScope?.(ast) || context.getScope(); // TODO: just use sourceCode.getScope() when we drop support for ESLint < v9.0.0
         contextIdentifiers = utils.getContextIdentifiers(
           sourceCode.scopeManager,
-          ast
+          ast,
         );
 
         const metaNode = ruleInfo.meta;
         const messagesNode =
           metaNode &&
           metaNode.properties &&
           metaNode.properties.find(
-            (p) => p.type === 'Property' && utils.getKeyName(p) === 'messages'
+            (p) => p.type === 'Property' && utils.getKeyName(p) === 'messages',
           );
 
         if (!messagesNode) {

diff --git a/lib/rules/prefer-object-rule.js b/lib/rules/prefer-object-rule.js
@@ -57,7 +57,7 @@ module.exports = {
             ) {
               const openParenToken = sourceCode.getFirstToken(
                 ruleInfo.create,
-                (token) => token.type === 'Punctuator' && token.value === '('
+                (token) => token.type === 'Punctuator' && token.value === '(',
               );
 
               /* istanbul ignore if */
@@ -68,7 +68,7 @@ module.exports = {
 
               yield fixer.replaceTextRange(
                 [ruleInfo.create.range[0], openParenToken.range[0]],
-                '{create'
+                '{create',
               );
               yield fixer.insertTextAfter(ruleInfo.create, '}');
             } else if (ruleInfo.create.type === 'ArrowFunctionExpression') {

diff --git a/lib/rules/prefer-output-null.js b/lib/rules/prefer-output-null.js
@@ -49,7 +49,7 @@ module.exports = {
             function getTestInfo(key) {
               if (test.type === 'ObjectExpression') {
                 return test.properties.find(
-                  (item) => item.type === 'Property' && item.key.name === key
+                  (item) => item.type === 'Property' && item.key.name === key,
                 );
               }
               return null;

diff --git a/lib/rules/prefer-placeholders.js b/lib/rules/prefer-placeholders.js
@@ -66,7 +66,7 @@ module.exports = {
 
               const variable = findVariable(
                 scopeManager.acquire(messageNode) || scopeManager.globalScope,
-                messageNode
+                messageNode,
               );
 
               if (

diff --git a/lib/rules/prefer-replace-text.js b/lib/rules/prefer-replace-text.js
@@ -43,7 +43,7 @@ module.exports = {
       Program(ast) {
         contextIdentifiers = utils.getContextIdentifiers(
           sourceCode.scopeManager,
-          ast
+          ast,
         );
       },