[meta] fix some formatting

CHANGELOG.md

@@ -20,7 +20,7 @@ This change log adheres to standards from [Keep a CHANGELOG](http://keepachangel
 ### Fixed
 - [`no-duplicates`]: ensure autofix avoids excessive newlines ([#2028], thanks [@ertrzyiks])
 - [`extensions`]: avoid crashing on partially typed import/export statements ([#2118], thanks [@ljharb])
-- [`no-extraneous-dependencies`]: add ESM intermediate package.json support] ([#2121], thanks [@paztis])
+- [`no-extraneous-dependencies`]: add ESM intermediate package.json support ([#2121], thanks [@paztis])
 - Use `context.getPhysicalFilename()` when available (ESLint 7.28+) ([#2160], thanks [@pmcelhaney])
 - [`extensions`]/`importType`: fix isScoped treating @/abc as scoped module ([#2146], thanks [@rperello])
 

docs/rules/no-unresolved.md

@@ -6,7 +6,7 @@ as defined by standard Node `require.resolve` behavior.
 See [settings](../../README.md#settings) for customization options for the resolution (i.e.
 additional filetypes, `NODE_PATH`, etc.)
 
-This rule can also optionally report on unresolved modules in CommonJS `require('./foo')` calls and AMD `require(['./foo'], function (foo){...})` and `define(['./foo'], function (foo){...})`.
+This rule can also optionally report on unresolved modules in CommonJS `require('./foo')` calls and AMD `require(['./foo'], function (foo) {...})` and `define(['./foo'], function (foo) {...})`.
 
 To enable this, send `{ commonjs: true/false, amd: true/false }` as a rule option.
 Both are disabled by default.

resolvers/webpack/index.js

@@ -118,8 +118,7 @@ exports.resolve = function (source, file, settings) {
 
     if (typeof configIndex !== 'undefined' && webpackConfig.length > configIndex) {
       webpackConfig = webpackConfig[configIndex];
-    }
-    else {
+    } else {
       webpackConfig = find(webpackConfig, function findFirstWithResolve(config) {
         return !!config.resolve;
       });
@@ -311,16 +310,16 @@ function createWebpack1ResolveSync(webpackRequire, resolveConfig, plugins) {
 /* eslint-disable */
 // from https://github.com/webpack/webpack/blob/v1.13.0/lib/WebpackOptionsApply.js#L365
 function makeRootPlugin(ModulesInRootPlugin, name, root) {
-  if(typeof root === "string")
+  if (typeof root === 'string') {
     return new ModulesInRootPlugin(name, root);
-  else if(Array.isArray(root)) {
+  } else if (Array.isArray(root)) {
     return function() {
-      root.forEach(function(root) {
+      root.forEach(function (root) {
         this.apply(new ModulesInRootPlugin(name, root));
       }, this);
     };
   }
-  return function() {};
+  return function () {};
 }
 /* eslint-enable */
 
@@ -436,17 +435,17 @@ function findConfigPath(configPath, packageDir) {
 }
 
 function registerCompiler(moduleDescriptor) {
-  if(moduleDescriptor) {
-    if(typeof moduleDescriptor === 'string') {
+  if (moduleDescriptor) {
+    if (typeof moduleDescriptor === 'string') {
       require(moduleDescriptor);
-    } else if(!Array.isArray(moduleDescriptor)) {
+    } else if (!Array.isArray(moduleDescriptor)) {
       moduleDescriptor.register(require(moduleDescriptor.module));
     } else {
-      for(let i = 0; i < moduleDescriptor.length; i++) {
+      for (let i = 0; i < moduleDescriptor.length; i++) {
         try {
           registerCompiler(moduleDescriptor[i]);
           break;
-        } catch(e) {
+        } catch (e) {
           log('Failed to register compiler for moduleDescriptor[]:', i, moduleDescriptor);
         }
       }

src/rules/dynamic-import-chunkname.js

@@ -69,7 +69,7 @@ module.exports = {
 
         try {
           // just like webpack itself does
-          vm.runInNewContext(`(function(){return {${comment.value}}})()`);
+          vm.runInNewContext(`(function() {return {${comment.value}}})()`);
         }
         catch (error) {
           context.report({

src/rules/first.js

@@ -42,7 +42,7 @@ module.exports = {
         const errorInfos = [];
         let shouldSort = true;
         let lastSortNodesIndex = 0;
-        body.forEach(function (node, index){
+        body.forEach(function (node, index) {
           if (!anyExpressions && isPossibleDirective(node)) {
             return;
           }

src/rules/no-unresolved.js

@@ -15,35 +15,36 @@ module.exports = {
       url: docsUrl('no-unresolved'),
     },
 
-    schema: [ makeOptionsSchema({
-      caseSensitive: { type: 'boolean', default: true },
-    })],
+    schema: [
+      makeOptionsSchema({
+        caseSensitive: { type: 'boolean', default: true },
+      }),
+    ],
   },
 
   create: function (context) {
-
     function checkSourceValue(source) {
-      const shouldCheckCase = !CASE_SENSITIVE_FS &&
-        (!context.options[0] || context.options[0].caseSensitive !== false);
+      const shouldCheckCase = !CASE_SENSITIVE_FS
+        && (!context.options[0] || context.options[0].caseSensitive !== false);
 
       const resolvedPath = resolve(source.value, context);
 
       if (resolvedPath === undefined) {
-        context.report(source,
-          `Unable to resolve path to module '${source.value}'.`);
-      }
-
-      else if (shouldCheckCase) {
+        context.report(
+          source,
+          `Unable to resolve path to module '${source.value}'.`
+        );
+      } else if (shouldCheckCase) {
         const cacheSettings = ModuleCache.getSettings(context.settings);
         if (!fileExistsWithCaseSync(resolvedPath, cacheSettings)) {
-          context.report(source,
-            `Casing of ${source.value} does not match the underlying filesystem.`);
+          context.report(
+            source,
+            `Casing of ${source.value} does not match the underlying filesystem.`
+          );
         }
-
       }
     }
 
     return moduleVisitor(checkSourceValue, context.options[0]);
-
   },
 };

src/rules/no-unused-modules.js

@@ -540,7 +540,7 @@ module.exports = {
 
       const value = exportsKey === IMPORT_DEFAULT_SPECIFIER ? DEFAULT : exportsKey;
 
-      if (typeof exportStatement !== 'undefined'){
+      if (typeof exportStatement !== 'undefined') {
         if (exportStatement.whereUsed.size < 1) {
           context.report(
             node,

src/rules/order.js

@@ -47,8 +47,7 @@ function takeTokensAfterWhile(sourceCode, node, condition) {
   for (let i = 0; i < tokens.length; i++) {
     if (condition(tokens[i])) {
       result.push(tokens[i]);
-    }
-    else {
+    } else {
       break;
     }
   }
@@ -61,8 +60,7 @@ function takeTokensBeforeWhile(sourceCode, node, condition) {
   for (let i = tokens.length - 1; i >= 0; i--) {
     if (condition(tokens[i])) {
       result.push(tokens[i]);
-    }
-    else {
+    } else {
       break;
     }
   }

src/rules/prefer-default-export.js

@@ -69,8 +69,7 @@ module.exports = {
           node.declaration.declarations.forEach(function(declaration) {
             captureDeclaration(declaration.id);
           });
-        }
-        else {
+        } else {
           // captures 'export function foo() {}' syntax
           specifierExportCount++;
         }

tests/files/foo-bar-resolver-no-version.js

@@ -5,10 +5,8 @@ exports.resolveImport = function (modulePath, sourceFile, config) {
   if (sourceFileName === 'foo.js') {
     return path.join(__dirname, 'bar.jsx')
   }
-  else if (sourceFileName === 'exception.js') {
+  if (sourceFileName === 'exception.js') {
     throw new Error('foo-bar-resolver-v1 resolveImport test exception')
   }
-  else {
-    return undefined
-  }
+  return undefined;
 }

tests/files/foo-bar-resolver-v1.js

@@ -3,14 +3,12 @@ var path = require('path')
 exports.resolveImport = function (modulePath, sourceFile, config) {
   var sourceFileName = path.basename(sourceFile)
   if (sourceFileName === 'foo.js') {
-    return path.join(__dirname, 'bar.jsx')
+    return path.join(__dirname, 'bar.jsx');
   }
-  else if (sourceFileName === 'exception.js') {
-    throw new Error('foo-bar-resolver-v1 resolveImport test exception')
+  if (sourceFileName === 'exception.js') {
+    throw new Error('foo-bar-resolver-v1 resolveImport test exception');
   }
-  else {
-    return undefined
-  }
-}
+  return undefined;
+};
 
-exports.interfaceVersion = 1
+exports.interfaceVersion = 1;

tests/files/foo-bar-resolver-v2.js

@@ -5,12 +5,10 @@ exports.resolve = function (modulePath, sourceFile, config) {
   if (sourceFileName === 'foo.js') {
     return { found: true, path: path.join(__dirname, 'bar.jsx') }
   }
-  else if (sourceFileName === 'exception.js') {
+  if (sourceFileName === 'exception.js') {
     throw new Error('foo-bar-resolver-v2 resolve test exception')
   }
-  else {
-    return { found: false }
-  }
-}
+  return { found: false };
+};
 
-exports.interfaceVersion = 2
+exports.interfaceVersion = 2;

tests/files/typescript.ts

@@ -23,14 +23,14 @@ export function getFoo() : MyType {
 }
 
 export module MyModule {
-  export function ModuleFunction(){}
+  export function ModuleFunction() {}
 }
 
 export namespace MyNamespace {
-  export function NamespaceFunction(){}
+  export function NamespaceFunction() {}
 
   export module NSModule {
-    export function NSModuleFunction(){}
+    export function NSModuleFunction() {}
   }
 }
 

tests/src/rules/newline-after-import.js

@@ -26,7 +26,7 @@ ruleTester.run('newline-after-import', require('rules/newline-after-import'), {
       code: `const x = () => require('baz') && require('bar')`,
       parserOptions: { ecmaVersion: 6 } ,
     },
-    `function x(){ require('baz'); }`,
+    `function x() { require('baz'); }`,
     `a(require('b'), require('c'), require('d'));`,
     `function foo() {
       switch (renderData.modalViewKey) {

tests/src/rules/no-cycle.js

@@ -61,7 +61,7 @@ ruleTester.run('no-cycle', rule, {
         options: [{ maxDepth: 1 }],
       }),
       test({
-        code: `import("./${testDialect}/depth-two").then(function({ foo }){})`,
+        code: `import("./${testDialect}/depth-two").then(function({ foo }) {})`,
         options: [{ maxDepth: 1 }],
         parser: require.resolve('babel-eslint'),
       }),

tests/src/rules/no-unresolved.js

@@ -122,7 +122,7 @@ function runResolverTests(resolver) {
         type: 'Literal',
         }] }),
       rest({
-        code: "import('in-alternate-root').then(function({DEEP}){});",
+        code: "import('in-alternate-root').then(function({DEEP}) {});",
         errors: [{
           message: 'Unable to resolve path to module \'in-alternate-root\'.',
           type: 'Literal',
@@ -138,7 +138,7 @@ function runResolverTests(resolver) {
 
       // check with eslint parser
       testVersion('>= 7', () => rest({
-        code: "import('in-alternate-root').then(function({DEEP}){});",
+        code: "import('in-alternate-root').then(function({DEEP}) {});",
         errors: [{
           message: 'Unable to resolve path to module \'in-alternate-root\'.',
           type: 'Literal',

tests/src/rules/no-useless-path-segments.js

@@ -32,7 +32,7 @@ function runResolverTests(resolver) {
         parser: require.resolve('babel-eslint') }),
       test({ code: 'import("..")',
         parser: require.resolve('babel-eslint') }),
-      test({ code: 'import("fs").then(function(fs){})',
+      test({ code: 'import("fs").then(function(fs) {})',
         parser: require.resolve('babel-eslint') }),
     ],
 

utils/moduleVisitor.js

@@ -131,7 +131,7 @@ function makeOptionsSchema(additionalProperties) {
     'additionalProperties': false,
   };
 
-  if (additionalProperties){
+  if (additionalProperties) {
     for (const key in additionalProperties) {
       base.properties[key] = additionalProperties[key];
     }