feat(@schematics/update): add doc for --force flag

hansl · hansl · commit 20a81684a845 · 2018-06-05T17:55:16.000-07:00
And show packages that have invalid deps when --force, but as a warning.
diff --git a/docs/specifications/update.md b/docs/specifications/update.md
@@ -16,6 +16,7 @@ You can specify more than one package. Each package follows the convention of `[
 | Flag | Argument | Description |
 |---|---|---|
 | `--all` | `boolean` | If true, implies that all dependencies should be updated. Defaults is false, using dependencies from the command line instead. |
+| `--force` | `boolean` | If true, skip the verification step and perform the update even if some peer dependencies would be invalidated. Peer dependencies errors will still be shown as warning. Defaults to false. |
 | `--next` | `boolean` | If true, allows version discovery to include Beta and RC. Defaults to false. |
 | `--migration-only` | `boolean` | If true, don't change the `package.json` file, only apply migration scripts. |
 | `--from` | `version` | Apply migrations from a certain version number. |
diff --git a/packages/schematics/update/update/index.ts b/packages/schematics/update/update/index.ts
@@ -113,9 +113,9 @@ function _validateReversePeerDependencies(
 
 function _validateUpdatePackages(
   infoMap: Map<string, PackageInfo>,
+  force: boolean,
   logger: logging.LoggerApi,
 ): void {
-  logger = logger.createChild('validateUpdate');
   logger.debug('Updating the following packages:');
   infoMap.forEach(info => {
     if (info.target) {
@@ -140,7 +140,7 @@ function _validateUpdatePackages(
       || peerErrors;
   });
 
-  if (peerErrors) {
+  if (!force && peerErrors) {
     throw new SchematicsException(`Incompatible peer dependencies found. See above.`);
   }
 }
@@ -619,9 +619,12 @@ export default function(options: UpdateSchema): Rule {
       switchMap(infoMap => {
         // Now that we have all the information, check the flags.
         if (packages.size > 0) {
-          if (!options.force) {
-            _validateUpdatePackages(infoMap, logger);
-          }
+          const sublog = new logging.LevelCapLogger(
+            'validation',
+            logger.createChild(''),
+            'warn',
+          );
+          _validateUpdatePackages(infoMap, options.force, sublog);
 
           return _performUpdate(tree, context, infoMap, logger);
         } else {
