refactor: remove unused local variables

Author: alan-agius4

packages/angular/cli/src/command-builder/command-runner.ts

@@ -32,7 +32,7 @@ import { colors } from '../utilities/color';
 import { AngularWorkspace, getWorkspace } from '../utilities/config';
 import { assertIsError } from '../utilities/error';
 import { PackageManagerUtils } from '../utilities/package-manager';
-import { CommandContext, CommandModuleError, CommandScope } from './command-module';
+import { CommandContext, CommandModuleError } from './command-module';
 import { addCommandModuleToYargs, demandCommandFailureMessage } from './utilities/command';
 import { jsonHelpUsage } from './utilities/json-help';
 import { normalizeOptionsMiddleware } from './utilities/normalize-options-middleware';

packages/angular/cli/src/utilities/color.ts

@@ -9,8 +9,6 @@
 import * as ansiColors from 'ansi-colors';
 import { WriteStream } from 'tty';
 
-type AnsiColors = typeof ansiColors;
-
 function supportColor(): boolean {
   if (process.env.FORCE_COLOR !== undefined) {
     // 2 colors: FORCE_COLOR = 0 (Disables colors), depth 1

packages/angular_devkit/build_angular/src/builders/browser-esbuild/compiler-plugin.ts

@@ -260,7 +260,7 @@ export function createCompilerPlugin(
         // Create the Angular specific program that contains the Angular compiler
         const angularProgram = new compilerCli.NgtscProgram(rootNames, compilerOptions, host);
         const angularCompiler = angularProgram.compiler;
-        const { ignoreForDiagnostics, ignoreForEmit } = angularCompiler;
+        const { ignoreForDiagnostics } = angularCompiler;
         const typeScriptProgram = angularProgram.getTsProgram();
 
         const builder = ts.createAbstractBuilder(typeScriptProgram, host);

packages/angular_devkit/build_angular/src/utils/color.ts

@@ -9,8 +9,6 @@
 import * as ansiColors from 'ansi-colors';
 import { WriteStream } from 'tty';
 
-type AnsiColors = typeof ansiColors;
-
 function supportColor(): boolean {
   if (process.env.FORCE_COLOR !== undefined) {
     // 2 colors: FORCE_COLOR = 0 (Disables colors), depth 1

packages/angular_devkit/build_angular/src/utils/purge-cache.ts

@@ -7,7 +7,7 @@
  */
 
 import { BuilderContext } from '@angular-devkit/architect';
-import { PathLike, existsSync, promises as fsPromises } from 'fs';
+import { existsSync, promises as fsPromises } from 'fs';
 import { join } from 'path';
 import { normalizeCacheOptions } from './normalize-cache';
 

packages/angular_devkit/build_angular/src/webpack/configs/dev-server.ts

@@ -11,12 +11,7 @@ import { existsSync, promises as fsPromises } from 'fs';
 import { extname, posix, resolve } from 'path';
 import { URL, pathToFileURL } from 'url';
 import { Configuration, RuleSetRule } from 'webpack';
-import type {
-  Configuration as DevServerConfiguration,
-  NextFunction,
-  Request,
-  Response,
-} from 'webpack-dev-server';
+import type { Configuration as DevServerConfiguration } from 'webpack-dev-server';
 import { WebpackConfigOptions, WebpackDevServerOptions } from '../../utils/build-options';
 import { assertIsError } from '../../utils/error';
 import { loadEsmModule } from '../../utils/load-esm';

packages/angular_devkit/core/node/testing/index.ts

@@ -9,7 +9,7 @@
 import * as fs from 'fs';
 import * as os from 'os';
 import * as path from 'path';
-import { Path, PathFragment, getSystemPath, join, normalize, virtualFs } from '../../src';
+import { Path, getSystemPath, join, normalize, virtualFs } from '../../src';
 import { NodeJsSyncHost } from '../host';
 
 /**

packages/angular_devkit/schematics/src/engine/interface.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import { analytics, logging } from '@angular-devkit/core';
+import { logging } from '@angular-devkit/core';
 import { Observable } from 'rxjs';
 import { Url } from 'url';
 import { FileEntry, MergeStrategy, Tree } from '../tree/interface';

packages/schematics/angular/module/index.ts

@@ -9,7 +9,6 @@
 import { Path, normalize } from '@angular-devkit/core';
 import {
   Rule,
-  SchematicsException,
   Tree,
   apply,
   applyTemplates,

scripts/json-help.ts

@@ -7,7 +7,7 @@
  */
 
 import { logging } from '@angular-devkit/core';
-import { spawn, spawnSync } from 'child_process';
+import { spawnSync } from 'child_process';
 import { promises as fs } from 'fs';
 import * as os from 'os';
 import { JsonHelp } from 'packages/angular/cli/src/command-builder/utilities/json-help';

tests/legacy-cli/e2e/utils/packages.ts

@@ -1,5 +1,5 @@
 import { getGlobalVariable } from './env';
-import { ProcessOutput, npm, silentNpm, silentYarn } from './process';
+import { ProcessOutput, silentNpm, silentYarn } from './process';
 
 export function getActivePackageManager(): 'npm' | 'yarn' {
   const value = getGlobalVariable('package-manager');
@@ -49,8 +49,6 @@ export async function setRegistry(useTestRegistry: boolean): Promise<void> {
     ? getGlobalVariable('package-registry')
     : 'https://registry.npmjs.org';
 
-  const isCI = getGlobalVariable('ci');
-
   // Ensure local test registry is used when outside a project
   // Yarn supports both `NPM_CONFIG_REGISTRY` and `YARN_REGISTRY`.
   process.env['NPM_CONFIG_REGISTRY'] = url;