feat(@angular/cli): disable progress when running outside TTY

The default value is changed from `true` to `progress.stdout.isTTY`. It
still has lower priority than value specified by command line flag or in
the .angular-cli.json config.

Fixes #8148

Author: devoto13

docs/documentation/build.md

@@ -261,7 +261,7 @@ Note: service worker support is experimental and subject to change.
 <details>
   <summary>progress</summary>
   <p>
-    <code>--progress</code> (aliases: <code>-pr</code>) <em>default value: true</<em>
+    <code>--progress</code> (aliases: <code>-pr</code>) <em>default value: true inside TTY, false otherwise</<em>
   </p>
   <p>
     Log progress to the console while building.

docs/documentation/eject.md

@@ -165,7 +165,7 @@ ng eject
 <details>
   <summary>progress</summary>
   <p>
-    <code>--progress</code> (aliases: <code>-pr</code>) <em>default value: true</em>
+    <code>--progress</code> (aliases: <code>-pr</code>) <em>default value: true inside TTY, false otherwise</em>
   </p>
   <p>
     Log progress to the console while building.

docs/documentation/serve.md

@@ -234,7 +234,7 @@ All the build Options are available in serve, below are the additional options.
 <details>
   <summary>progress</summary>
   <p>
-    <code>--progress</code> (aliases: <code>-pr</code>) <em>default value: true</em>
+    <code>--progress</code> (aliases: <code>-pr</code>) <em>default value: true inside TTY, false otherwise</em>
   </p>
   <p>
     Log progress to the console while building.

docs/documentation/test.md

@@ -109,7 +109,7 @@ You can run tests with coverage via `--code-coverage`. The coverage report will
 <details>
   <summary>progress</summary>
   <p>
-    <code>--progress</code> <em>default value: true</em>
+    <code>--progress</code> <em>default value: true inside TTY, false otherwise</em>
   </p>
   <p>
     Log progress to the console while in progress.

docs/documentation/xi18n.md

@@ -59,7 +59,7 @@
 <details>
   <summary>progress</summary>
   <p>
-    <code>--progress</code>
+    <code>--progress</code> <em>default value: true inside TTY, false otherwise</<em>
   </p>
   <p>
     Log progress to the console while running.

packages/@angular/cli/commands/build.ts

@@ -83,7 +83,9 @@ export const baseBuildCommandOptions: any = [
     type: Boolean,
     aliases: ['pr'],
     description: 'Log progress to the console while building.',
-    default: buildConfigDefaults['progress']
+    default: typeof buildConfigDefaults['progress'] !== 'undefined'
+      ? buildConfigDefaults['progress']
+      : process.stdout.isTTY === true
   },
   {
     name: 'i18n-file',

packages/@angular/cli/commands/test.ts

@@ -64,8 +64,10 @@ const TestCommand = Command.extend({
     {
       name: 'progress',
       type: Boolean,
-      default: testConfigDefaults['progress'],
-      description: 'Log progress to the console while in progress.'
+      description: 'Log progress to the console while in progress.',
+      default: typeof testConfigDefaults['progress'] !== 'undefined'
+        ? testConfigDefaults['progress']
+        : process.stdout.isTTY === true,
     },
     {
       name: 'browsers',

packages/@angular/cli/commands/xi18n.ts

@@ -36,8 +36,8 @@ const Xi18nCommand = Command.extend({
     {
       name: 'progress',
       type: Boolean,
-      default: true,
-      description: 'Log progress to the console while running.'
+      description: 'Log progress to the console while running.',
+      default: process.stdout.isTTY === true,
     },
     {
       name: 'app',

packages/@angular/cli/lib/config/schema.json

@@ -476,8 +476,7 @@
             },
             "progress": {
               "description": "The ssl key used by the server.",
-              "type": "boolean",
-              "default": true
+              "type": "boolean"
             },
             "poll": {
               "description": "Enable and define the file watching poll time period (milliseconds).",