Merge pull request #2792 from nestjs/fix/type-check-error-2791

fix: show type check warning only when builder is not swc

Author: kamilmysliwiec

Diff for: actions/build.action.ts

@@ -3,14 +3,14 @@ import { join } from 'path';
 import * as ts from 'typescript';
 import { Input } from '../commands';
 import { AssetsManager } from '../lib/compiler/assets-manager';
+import { deleteOutDirIfEnabled } from '../lib/compiler/helpers/delete-out-dir';
 import { getBuilder } from '../lib/compiler/helpers/get-builder';
 import { getTscConfigPath } from '../lib/compiler/helpers/get-tsc-config.path';
 import { getValueOrDefault } from '../lib/compiler/helpers/get-value-or-default';
 import { getWebpackConfigPath } from '../lib/compiler/helpers/get-webpack-config-path';
 import { TsConfigProvider } from '../lib/compiler/helpers/tsconfig-provider';
 import { PluginsLoader } from '../lib/compiler/plugins/plugins-loader';
 import { TypeScriptBinaryLoader } from '../lib/compiler/typescript-loader';
-import { deleteOutDirIfEnabled } from '../lib/compiler/helpers/delete-out-dir';
 import {
   Configuration,
   ConfigurationLoader,
@@ -21,7 +21,7 @@ import {
   defaultWebpackConfigFilename,
 } from '../lib/configuration/defaults';
 import { FileSystemReader } from '../lib/readers';
-import { ERROR_PREFIX } from '../lib/ui';
+import { ERROR_PREFIX, INFO_PREFIX } from '../lib/ui';
 import { isModuleAvailable } from '../lib/utils/is-module-available';
 import { AbstractAction } from './abstract.action';
 import webpack = require('webpack');
@@ -109,6 +109,20 @@ export class BuildAction extends AbstractAction {
       watchAssetsMode,
     );
 
+    const typeCheck = getValueOrDefault<boolean>(
+      configuration,
+      'compilerOptions.typeCheck',
+      appName,
+      'typeCheck',
+      commandOptions,
+    );
+    if (typeCheck && builder.type !== 'swc') {
+      console.warn(
+        INFO_PREFIX +
+          ` "typeCheck" will not have any effect when "builder" is not "swc".`,
+      );
+    }
+
     switch (builder.type) {
       case 'tsc':
         return this.runTsc(

Diff for: commands/build.command.ts

@@ -1,5 +1,5 @@
 import { Command, CommanderStatic } from 'commander';
-import { ERROR_PREFIX, INFO_PREFIX } from '../lib/ui';
+import { ERROR_PREFIX } from '../lib/ui';
 import { AbstractCommand } from './abstract.command';
 import { Input } from './command.input';
 
@@ -60,12 +60,6 @@ export class BuildCommand extends AbstractCommand {
           value: command.builder,
         });
 
-        if (command.typeCheck && command.builder !== 'swc') {
-          console.warn(
-            INFO_PREFIX +
-              ` "typeCheck" will not have any effect when "builder" is not "swc".`,
-          );
-        }
         options.push({
           name: 'typeCheck',
           value: command.typeCheck,

Diff for: commands/start.command.ts

@@ -1,5 +1,5 @@
 import { Command, CommanderStatic } from 'commander';
-import { ERROR_PREFIX, INFO_PREFIX } from '../lib/ui';
+import { ERROR_PREFIX } from '../lib/ui';
 import { getRemainingFlags } from '../lib/utils/remaining-flags';
 import { AbstractCommand } from './abstract.command';
 import { Input } from './command.input';
@@ -95,12 +95,6 @@ export class StartCommand extends AbstractCommand {
           value: command.builder,
         });
 
-        if (command.typeCheck && command.builder !== 'swc') {
-          console.warn(
-            INFO_PREFIX +
-              ` "typeCheck" will not have any effect when "builder" is not "swc".`,
-          );
-        }
         options.push({
           name: 'typeCheck',
           value: command.typeCheck,