refactor(@angular-devkit/schematics): minor rule cleanup (#14313)

Author: clydin

packages/angular_devkit/schematics/src/rules/base.ts

@@ -70,7 +70,7 @@ export function mergeWith(source: Source, strategy: MergeStrategy = MergeStrateg
 
 
 export function noop(): Rule {
-  return (tree: Tree, _context: SchematicContext) => tree;
+  return () => {};
 }
 
 
@@ -159,12 +159,9 @@ export function forEach(operator: FileOperator): Rule {
         tree.overwrite(newEntry.path, newEntry.content);
       }
     });
-
-    return tree;
   };
 }
 
-
 export function composeFileOperators(operators: FileOperator[]): FileOperator {
   return (entry: FileEntry) => {
     let current: FileEntry | null = entry;

packages/angular_devkit/schematics/src/rules/call.ts

@@ -7,7 +7,7 @@
  */
 import { BaseException, isObservable, isPromise } from '@angular-devkit/core';
 import { Observable, from, of as observableOf, throwError } from 'rxjs';
-import { defaultIfEmpty, last, map, mergeMap, tap } from 'rxjs/operators';
+import { defaultIfEmpty, last, mergeMap, tap } from 'rxjs/operators';
 import { Rule, SchematicContext, Source } from '../engine/interface';
 import { Tree, TreeSymbol } from '../tree/interface';