Skip to content

Commit abd2f97

Browse files
clydinKeen Yee Liau
authored and
Keen Yee Liau
committed
fix(@schematics/update): remove update all suggestion message
The language within the message specifies that update all will _try_ to update all the packages. In practice and especially for larger projects, this operation will mostly likely not succeed. This can lead to the users attempting to use the all option with the force option which has a decent probability to break the project via incorrect peer dependency installation or other incompability version increases (packages that don't follow semver, for instance). Reference: #14561 (comment)
1 parent ebbfbd6 commit abd2f97

File tree

1 file changed

+0
-4
lines changed
  • packages/schematics/update/update

1 file changed

+0
-4
lines changed

packages/schematics/update/update/index.ts

-4
Original file line numberDiff line numberDiff line change
@@ -508,10 +508,6 @@ function _usageMessage(
508508
logger.info(' ' + fields.map((x, i) => x.padEnd(pads[i])).join(''));
509509
});
510510

511-
logger.info('\n');
512-
logger.info('There might be additional packages that are outdated.');
513-
logger.info('Run "ng update --all" to try to update all at the same time.\n');
514-
515511
return of<void>(undefined);
516512
}
517513

0 commit comments

Comments
 (0)