feat(@ngtools/webpack): deprecate discoverLazyRoutes, additionalLazyModules and additionalLazyModuleResources options

String form of lazy loading and SystemJsNgModuleLoader are deprecated, and this is part of its usage.

See: https://angular.io/api/core/SystemJsNgModuleLoader#systemjsngmoduleloader

Author: Alan

packages/ngtools/webpack/src/interfaces.ts

@@ -45,14 +45,22 @@ export interface AngularCompilerPluginOptions {
   logger?: logging.Logger;
   directTemplateLoading?: boolean;
 
-  // When using the loadChildren string syntax, @ngtools/webpack must query @angular/compiler-cli
-  // via a private API to know which lazy routes exist. This increases build and rebuild time.
-  // When using Ivy, the string syntax is not supported at all. Thus we shouldn't attempt that.
-  // This option is also used for when the compilation doesn't need this sort of processing at all.
+  /**
+   * When using the loadChildren string syntax, @ngtools/webpack must query @angular/compiler-cli
+   * via a private API to know which lazy routes exist. This increases build and rebuild time.
+   * When using Ivy, the string syntax is not supported at all. Thus we shouldn't attempt that
+   * This option is also used for when the compilation doesn't need this sort of processing at all.
+   * @deprecated SystemJsNgModuleLoader is deprecated, and this is part of its usage.
+   */
   discoverLazyRoutes?: boolean;
 
-  // added to the list of lazy routes
+  /** added to the list of lazy routes
+   * @deprecated SystemJsNgModuleLoader is deprecated, and this is part of its usage.
+   */
   additionalLazyModules?: { [module: string]: string };
+  /**
+   * @deprecated SystemJsNgModuleLoader is deprecated, and this is part of its usage.
+   */
   additionalLazyModuleResources?: string[];
 
   // The ContextElementDependency of correct Webpack compilation.