@rushstack/eslint-config 2.5.1 → 2.5.2

package/profile/_macros.js

@@ -1,102 +1,102 @@
-// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
-// See LICENSE in the project root for license information.
-
-// This is a workaround for the @typescript-eslint/naming-convention rule, whose options currently
-// support a "selector" field that cannot match multiple selectors. This function receives an input
-// array such as:
-//
-// [
-//   {
-//     selectors: ['class', 'typeAlias', 'enum'],
-//     format: ['PascalCase']
-//   },
-//   . . .
-// ]
-//
-// ...and transforms "selectors" -> "selector, returning an array with expanded entries like this:
-//
-// [
-//   {
-//     selector: 'class',
-//     format: ['PascalCase']
-//   },
-//   {
-//     selector: 'typeAlias',
-//     format: ['PascalCase']
-//   },
-//   {
-//     selector: 'enum',
-//     format: ['PascalCase']
-//   },
-//   . . .
-// ]
-//
-// It also supports a "enforceLeadingUnderscoreWhenPrivate" macro that expands this:
-//
-// [
-//   {
-//     selectors: ['property'],
-//     enforceLeadingUnderscoreWhenPrivate: true,
-//     format: ['camelCase']
-//   },
-//   . . .
-// ]
-//
-// ...to produce this:
-//
-// [
-//   {
-//     selector: 'property',
-//
-//     leadingUnderscore: 'allow',
-//     format: ['camelCase']
-//   },
-//   {
-//     selector: 'property',
-//     modifiers: ['private'],
-//
-//     leadingUnderscore: 'require',
-//     format: ['camelCase']
-//   },
-//   . . .
-// ]
-function expandNamingConventionSelectors(inputBlocks) {
-  const firstPassBlocks = [];
-
-  // Expand "selectors" --> "selector"
-  for (const block of inputBlocks) {
-    for (const selector of block.selectors) {
-      const expandedBlock = { ...block };
-      delete expandedBlock.selectors;
-      expandedBlock.selector = selector;
-      firstPassBlocks.push(expandedBlock);
-    }
-  }
-
-  // Expand "enforceLeadingUnderscoreWhenPrivate" --> "leadingUnderscore"
-  const secondPassBlocks = [];
-  for (const block of firstPassBlocks) {
-    if (block.enforceLeadingUnderscoreWhenPrivate) {
-      const expandedBlock1 = {
-        ...block,
-        leadingUnderscore: 'allow'
-      };
-      delete expandedBlock1.enforceLeadingUnderscoreWhenPrivate;
-      secondPassBlocks.push(expandedBlock1);
-
-      const expandedBlock2 = {
-        ...block,
-        modifiers: ['private'],
-        leadingUnderscore: 'require'
-      };
-      delete expandedBlock2.enforceLeadingUnderscoreWhenPrivate;
-      secondPassBlocks.push(expandedBlock2);
-    } else {
-      secondPassBlocks.push(block);
-    }
-  }
-
-  return secondPassBlocks;
-}
-
-exports.expandNamingConventionSelectors = expandNamingConventionSelectors;
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
+
+// This is a workaround for the @typescript-eslint/naming-convention rule, whose options currently
+// support a "selector" field that cannot match multiple selectors. This function receives an input
+// array such as:
+//
+// [
+//   {
+//     selectors: ['class', 'typeAlias', 'enum'],
+//     format: ['PascalCase']
+//   },
+//   . . .
+// ]
+//
+// ...and transforms "selectors" -> "selector, returning an array with expanded entries like this:
+//
+// [
+//   {
+//     selector: 'class',
+//     format: ['PascalCase']
+//   },
+//   {
+//     selector: 'typeAlias',
+//     format: ['PascalCase']
+//   },
+//   {
+//     selector: 'enum',
+//     format: ['PascalCase']
+//   },
+//   . . .
+// ]
+//
+// It also supports a "enforceLeadingUnderscoreWhenPrivate" macro that expands this:
+//
+// [
+//   {
+//     selectors: ['property'],
+//     enforceLeadingUnderscoreWhenPrivate: true,
+//     format: ['camelCase']
+//   },
+//   . . .
+// ]
+//
+// ...to produce this:
+//
+// [
+//   {
+//     selector: 'property',
+//
+//     leadingUnderscore: 'allow',
+//     format: ['camelCase']
+//   },
+//   {
+//     selector: 'property',
+//     modifiers: ['private'],
+//
+//     leadingUnderscore: 'require',
+//     format: ['camelCase']
+//   },
+//   . . .
+// ]
+function expandNamingConventionSelectors(inputBlocks) {
+  const firstPassBlocks = [];
+
+  // Expand "selectors" --> "selector"
+  for (const block of inputBlocks) {
+    for (const selector of block.selectors) {
+      const expandedBlock = { ...block };
+      delete expandedBlock.selectors;
+      expandedBlock.selector = selector;
+      firstPassBlocks.push(expandedBlock);
+    }
+  }
+
+  // Expand "enforceLeadingUnderscoreWhenPrivate" --> "leadingUnderscore"
+  const secondPassBlocks = [];
+  for (const block of firstPassBlocks) {
+    if (block.enforceLeadingUnderscoreWhenPrivate) {
+      const expandedBlock1 = {
+        ...block,
+        leadingUnderscore: 'allow'
+      };
+      delete expandedBlock1.enforceLeadingUnderscoreWhenPrivate;
+      secondPassBlocks.push(expandedBlock1);
+
+      const expandedBlock2 = {
+        ...block,
+        modifiers: ['private'],
+        leadingUnderscore: 'require'
+      };
+      delete expandedBlock2.enforceLeadingUnderscoreWhenPrivate;
+      secondPassBlocks.push(expandedBlock2);
+    } else {
+      secondPassBlocks.push(block);
+    }
+  }
+
+  return secondPassBlocks;
+}
+
+exports.expandNamingConventionSelectors = expandNamingConventionSelectors;

package/profile/node-trusted-tool.js

@@ -1,18 +1,18 @@
-// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
-// See LICENSE in the project root for license information.
-
-// This profile enables lint rules intended for a Node.js project whose inputs will always
-// come from a developer or other trusted source.  Most build system tasks are like this,
-// since they operate on exclusively files prepared by a developer.
-//
-// This profile disables certain security rules that would otherwise prohibit APIs that could
-// cause a denial-of-service by consuming too many resources, or which might interact with
-// the filesystem in unsafe ways.  Such activities are safe and commonplace for a trusted tool.
-//
-// DO NOT use this profile for a library project that might also be loaded by a Node.js service;
-// use "@rushstack/eslint-config/profiles/node" instead.
-
-const { buildRules } = require('./_common');
-
-const rules = buildRules('node-trusted-tool');
-module.exports = rules;
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
+
+// This profile enables lint rules intended for a Node.js project whose inputs will always
+// come from a developer or other trusted source.  Most build system tasks are like this,
+// since they operate on exclusively files prepared by a developer.
+//
+// This profile disables certain security rules that would otherwise prohibit APIs that could
+// cause a denial-of-service by consuming too many resources, or which might interact with
+// the filesystem in unsafe ways.  Such activities are safe and commonplace for a trusted tool.
+//
+// DO NOT use this profile for a library project that might also be loaded by a Node.js service;
+// use "@rushstack/eslint-config/profiles/node" instead.
+
+const { buildRules } = require('./_common');
+
+const rules = buildRules('node-trusted-tool');
+module.exports = rules;

package/profile/node.js

@@ -1,11 +1,11 @@
-// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
-// See LICENSE in the project root for license information.
-
-// This profile enables lint rules intended for a general Node.js project, typically a web service.
-// It enables security rules that assume the service could receive malicious inputs from an
-// untrusted user.  If that is not the case, consider using the "node-trusted-tool" profile instead.
-
-const { buildRules } = require('./_common');
-
-const rules = buildRules('node');
-module.exports = rules;
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
+
+// This profile enables lint rules intended for a general Node.js project, typically a web service.
+// It enables security rules that assume the service could receive malicious inputs from an
+// untrusted user.  If that is not the case, consider using the "node-trusted-tool" profile instead.
+
+const { buildRules } = require('./_common');
+
+const rules = buildRules('node');
+module.exports = rules;

package/profile/web-app.js

@@ -1,13 +1,13 @@
-// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
-// See LICENSE in the project root for license information.
-
-// This profile enables lint rules intended for a web application.  It enables security rules
-// that are relevant to web browser APIs such as DOM.
-//
-// Also use this profile if you are creating a library that can be consumed by both Node.js
-// and web applications.
-
-const { buildRules } = require('./_common');
-
-const rules = buildRules('web-app');
-module.exports = rules;
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
+
+// This profile enables lint rules intended for a web application.  It enables security rules
+// that are relevant to web browser APIs such as DOM.
+//
+// Also use this profile if you are creating a library that can be consumed by both Node.js
+// and web applications.
+
+const { buildRules } = require('./_common');
+
+const rules = buildRules('web-app');
+module.exports = rules;

package/react.js

@@ -1,8 +1,8 @@
-// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
-// See LICENSE in the project root for license information.
-
-throw new Error(
-  'The react.js entry point has moved. Please update your ESLint configuration to reference' +
-    ' "@rushstack/eslint-config/mixins/react" instead.' +
-    '\n\nSee the documentation for details: https://www.npmjs.com/package/@rushstack/eslint-config'
-);
+// Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
+// See LICENSE in the project root for license information.
+
+throw new Error(
+  'The react.js entry point has moved. Please update your ESLint configuration to reference' +
+    ' "@rushstack/eslint-config/mixins/react" instead.' +
+    '\n\nSee the documentation for details: https://www.npmjs.com/package/@rushstack/eslint-config'
+);