Suppress no-console rule violations.

Author: iclanton

apps/api-documenter/.eslintrc.js

@@ -3,5 +3,14 @@ require('eslint-config-local/patch/modern-module-resolution');
 
 module.exports = {
   extends: ['eslint-config-local/profile/node-trusted-tool', 'eslint-config-local/mixins/friendly-locals'],
-  parserOptions: { tsconfigRootDir: __dirname }
+  parserOptions: { tsconfigRootDir: __dirname },
+
+  overrides: [
+    {
+      files: ['*.ts', '*.tsx'],
+      rules: {
+        'no-console': 'off'
+      }
+    }
+  ]
 };

apps/api-extractor/.eslintrc.js

@@ -3,5 +3,14 @@ require('eslint-config-local/patch/modern-module-resolution');
 
 module.exports = {
   extends: ['eslint-config-local/profile/node-trusted-tool', 'eslint-config-local/mixins/friendly-locals'],
-  parserOptions: { tsconfigRootDir: __dirname }
+  parserOptions: { tsconfigRootDir: __dirname },
+
+  overrides: [
+    {
+      files: ['*.ts', '*.tsx'],
+      rules: {
+        'no-console': 'off'
+      }
+    }
+  ]
 };

apps/heft/src/startWithVersionSelector.ts

@@ -1,6 +1,8 @@
 // Copyright (c) Microsoft Corporation. All rights reserved. Licensed under the MIT license.
 // See LICENSE in the project root for license information.
 
+/* eslint-disable no-console */
+
 // NOTE: Since startWithVersionSelector.ts is loaded in the same process as start.ts, any dependencies that
 // we import here may become side-by-side versions.  We want to minimize any dependencies.
 import * as path from 'path';

apps/lockfile-explorer-web/src/App.tsx

@@ -26,6 +26,7 @@ export const App = (): JSX.Element => {
       dispatch(loadEntries(lockfile));
     }
     loadLockfileAsync().catch((e) => {
+      // eslint-disable-next-line no-console
       console.log(`Failed to read lockfile: ${e}`);
     });
   }, []);

apps/lockfile-explorer-web/src/components/ConnectionModal/index.tsx

@@ -31,6 +31,7 @@ export const ConnectionModal = (): JSX.Element | ReactNull => {
     setManualChecked(true);
     keepAliveAsync().catch((e) => {
       // Keep alive cannot fail
+      // eslint-disable-next-line no-console
       console.error(`Unexpected exception: ${e}`);
     });
   }, []);

apps/lockfile-explorer-web/src/containers/LockfileEntryDetailsView/index.tsx

@@ -57,6 +57,7 @@ export const LockfileEntryDetailsView = (): JSX.Element | ReactNull => {
     }
 
     loadPackageJson(selectedEntry?.referrers || []).catch((e) => {
+      // eslint-disable-next-line no-console
       console.error(`Failed to load referrers package.json: ${e}`);
     });
     if (selectedEntry) {
@@ -73,6 +74,7 @@ export const LockfileEntryDetailsView = (): JSX.Element | ReactNull => {
           logDiagnosticInfo('No resolved entry for dependency:', dependencyToTrace);
         }
       } else if (selectedEntry) {
+        // eslint-disable-next-line no-console
         console.log('dependency to trace: ', dependencyToTrace);
         setInspectDependency(dependencyToTrace);
 
@@ -110,6 +112,7 @@ export const LockfileEntryDetailsView = (): JSX.Element | ReactNull => {
                   // YAML file.  If not, either something is wrong with our algorithm, or else
                   // something has changed about how PNPM manages its "transitivePeerDependencies"
                   // field.
+                  // eslint-disable-next-line no-console
                   console.error(
                     'Error analyzing influencers: A referrer appears to be missing its "transitivePeerDependencies" field in the YAML file: ',
                     dependencyToTrace,

apps/lockfile-explorer-web/src/containers/PackageJsonViewer/index.tsx

@@ -41,6 +41,7 @@ export const PackageJsonViewer = (): JSX.Element => {
       setPnpmfile(pnpmfile);
     }
     loadPnpmFileAsync().catch((e) => {
+      // eslint-disable-next-line no-console
       console.error(`Failed to load project's pnpm file: ${e}`);
     });
   }, []);
@@ -60,10 +61,12 @@ export const PackageJsonViewer = (): JSX.Element => {
     if (selectedEntry) {
       if (selectedEntry.entryPackageName) {
         loadPackageDetailsAsync(selectedEntry.packageJsonFolderPath).catch((e) => {
+          // eslint-disable-next-line no-console
           console.error(`Failed to load project information: ${e}`);
         });
       } else {
         // This is used to develop the lockfile explorer application in case there is a mistake in our logic
+        // eslint-disable-next-line no-console
         console.log('The selected entry has no entry name: ', selectedEntry.entryPackageName);
       }
     }

apps/lockfile-explorer-web/src/helpers/logDiagnosticInfo.ts

@@ -5,6 +5,7 @@
 // problems with the algorithm.
 export const logDiagnosticInfo = (...args: string[]): void => {
   if (window.appContext.debugMode) {
+    // eslint-disable-next-line no-console
     console.log('Diagnostic: ', ...args);
   }
 };

apps/lockfile-explorer-web/src/parsing/LockfileDependency.ts

@@ -71,6 +71,7 @@ export class LockfileDependency {
         new Path(containingEntry.packageJsonFolderPath).concat(relativePath)
       );
       if (!rootRelativePath) {
+        // eslint-disable-next-line no-console
         console.error('No root relative path for dependency!', name);
         return;
       }
@@ -89,6 +90,7 @@ export class LockfileDependency {
         };
         this.entryId = 'Peer: ' + this.name;
       } else {
+        // eslint-disable-next-line no-console
         console.error('Peer dependencies info missing!', node);
       }
     } else {

apps/lockfile-explorer-web/src/parsing/LockfileEntry.ts

@@ -79,6 +79,7 @@ export class LockfileEntry {
       const packageName = new Path(rawEntryId).basename();
 
       if (!packageJsonFolderPath || !packageName) {
+        // eslint-disable-next-line no-console
         console.error('Could not construct path for entry: ', rawEntryId);
         return;
       }