refactor(material/schematics): do not print hammer migration message when updating to v13

The Hammer gesture migration runs only for v9 and v10 but the global
post migration messages are printed regardless of whether the update
runs for v9 or v10. This commit fixes that by providing the target
version to the `globalPostMigration` functions for migrations.

Author: devversion

src/cdk/schematics/ng-update/devkit-migration-rule.ts

@@ -109,7 +109,7 @@ export function createMigrationSchematicRule(
     // registered devkit migrations.
     migrations.forEach(m => {
       const actionResult = isDevkitMigration(m) && m.globalPostMigration !== undefined ?
-          m.globalPostMigration(tree, context) : null;
+          m.globalPostMigration(tree, targetVersion, context) : null;
       if (actionResult) {
         runPackageManager = runPackageManager || actionResult.runPackageManager;
       }

src/cdk/schematics/ng-update/devkit-migration.ts

@@ -9,16 +9,17 @@
 import {SchematicContext, Tree} from '@angular-devkit/schematics';
 import {ProjectDefinition} from '@angular-devkit/core/src/workspace';
 import {Constructor, Migration, PostMigrationAction} from '../update-tool/migration';
+import {TargetVersion} from '../update-tool/target-version';
 
 export type DevkitContext = {
   /** Devkit tree for the current migrations. Can be used to insert/remove files. */
-  tree: Tree,
+  tree: Tree;
   /** Name of the project the migrations run against. */
   projectName: string;
   /** Workspace project the migrations run against. */
-  project: ProjectDefinition,
+  project: ProjectDefinition;
   /** Whether the migrations run for a test target. */
-  isTestTarget: boolean,
+  isTestTarget: boolean;
 };
 
 export abstract class DevkitMigration<Data> extends Migration<Data, DevkitContext> {
@@ -35,7 +36,8 @@ export abstract class DevkitMigration<Data> extends Migration<Data, DevkitContex
    * migration result of all individual targets. e.g. removing HammerJS if it
    * is not needed in any project target.
    */
-  static globalPostMigration?(tree: Tree, context: SchematicContext): PostMigrationAction;
+  static globalPostMigration?(tree: Tree, targetVersion: TargetVersion,
+                              context: SchematicContext): PostMigrationAction;
 }
 
 export type DevkitMigrationCtor<Data> = Constructor<DevkitMigration<Data>> &

src/material/schematics/ng-update/migrations/hammer-gestures-v9/hammer-gestures-migration.ts

@@ -6,12 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {
-  join,
-  Path,
-  relative,
-  dirname
-} from '@angular-devkit/core';
+import {dirname, join, Path, relative} from '@angular-devkit/core';
 import {SchematicContext, Tree} from '@angular-devkit/schematics';
 import {
   addSymbolToNgModuleMetadata,
@@ -64,11 +59,11 @@ interface PackageJson {
 }
 
 export class HammerGesturesMigration extends DevkitMigration<null> {
-  // Only enable this rule if the migration targets v9 or v10 and is running for a non-test
-  // target. We cannot migrate test targets since they have a limited scope
-  // (in regards to source files) and therefore the HammerJS usage detection can be incorrect.
-  enabled =
-      (this.targetVersion === TargetVersion.V9 || this.targetVersion === TargetVersion.V10) &&
+  // The migration is enabled when v9 or v10 are targeted, but actual targets are only
+  // migrated if they are not test targets. We cannot migrate test targets since they have
+  // a limited scope, in regards to their source files, and therefore the HammerJS usage
+  // detection could be incorrect.
+  enabled = HammerGesturesMigration._isMigrationEnabled(this.targetVersion) &&
       !this.context.isTestTarget;
 
   private _printer = ts.createPrinter();
@@ -828,7 +823,13 @@ export class HammerGesturesMigration extends DevkitMigration<null> {
    * on the analysis of the individual targets. For example: we only remove Hammer
    * from the "package.json" if it is not used in *any* project target.
    */
-  static override globalPostMigration(tree: Tree, context: SchematicContext): PostMigrationAction {
+  static override globalPostMigration(tree: Tree, target: TargetVersion,
+                                      context: SchematicContext): PostMigrationAction {
+    // Skip printing any global messages when the migration rule is disabled.
+    if (!this._isMigrationEnabled(target)) {
+      return;
+    }
+
     // Always notify the developer that the Hammer v9 migration does not migrate tests.
     context.logger.info(
         '\n⚠  General notice: The HammerJS v9 migration for Angular Components is not able to ' +
@@ -867,6 +868,12 @@ export class HammerGesturesMigration extends DevkitMigration<null> {
     }
     return false;
   }
+
+  /** Gets whether the migration is enabled or not. */
+  private static _isMigrationEnabled(target: TargetVersion) {
+    // Only enable this rule if the migration targets v9 or v10.
+    return (target === TargetVersion.V9 || target === TargetVersion.V10);
+  }
 }
 
 /**

src/material/schematics/ng-update/migrations/theming-api-v12/theming-api-migration.ts

@@ -35,7 +35,8 @@ export class ThemingApiMigration extends DevkitMigration<null> {
   }
 
   /** Logs out the number of migrated files at the end of the migration. */
-  static override globalPostMigration(_tree: unknown, context: SchematicContext): void {
+  static override globalPostMigration(_tree: unknown, _targetVersion: TargetVersion,
+                                      context: SchematicContext): void {
     const count = ThemingApiMigration.migratedFileCount;
 
     if (count > 0) {