chore(lint): enable requireInternalWithUnderscore tslint check

modules/@angular/compiler/src/animation/animation_compiler.ts

@@ -123,6 +123,7 @@ class _AnimationBuilder implements AnimationAstVisitor {
     return this._callAnimateMethod(ast, startingStylesExpr, o.literalArr(keyframeExpressions));
   }
 
+  /** @internal */
   _visitEndStateAnimation(ast: AnimationStepAst,
                             context: _AnimationBuilderContext): o.Expression {
     var startingStylesExpr = ast.startingStyles.visit(this, context);
@@ -136,6 +137,7 @@ class _AnimationBuilder implements AnimationAstVisitor {
     return this._callAnimateMethod(ast, startingStylesExpr, keyframesExpr);
   }
 
+  /** @internal */
   _callAnimateMethod(ast: AnimationStepAst, startingStylesExpr, keyframesExpr) {
     return _ANIMATION_FACTORY_RENDERER_VAR.callMethod('animate', [
       _ANIMATION_FACTORY_ELEMENT_VAR,

modules/@angular/compiler/src/template_parser.ts

@@ -168,6 +168,7 @@ export class TemplateParser {
     return new TemplateParseResult(result, errors);
   }
 
+  /** @internal */
   _assertNoReferenceDuplicationOnTemplate(result:any[], errors:TemplateParseError[]):void {
     const existingReferences = [];
     result

modules/@angular/compiler_cli/src/codegen.ts

@@ -33,6 +33,8 @@ const PREAMBLE = `/**
  * This file is generated by the Angular 2 template compiler.
  * Do not edit.
  */
+ /* tslint:disable */
+
 `;
 
 export class CodeGenerator {

modules/@angular/core/private_export.ts

@@ -21,8 +21,8 @@ import * as provider_util from './src/di/provider_util';
 import * as console from './src/console';
 import {Provider} from './index';
 import * as reflection from './src/reflection/reflection';
-// tslint:disable-next-line
-import {Reflector} from './src/reflection/reflection';
+// We need to import this name separately from the above wildcard, because this symbol is exposed.
+import {Reflector} from './src/reflection/reflection'; // tslint:disable-line
 
 import {
   NoOpAnimationPlayer as NoOpAnimationPlayer_,

modules/@angular/core/src/animation/animation_player.ts

@@ -16,11 +16,13 @@ export abstract class AnimationPlayer {
 }
 
 export class NoOpAnimationPlayer implements AnimationPlayer {
+
   private _subscriptions = [];
   public parentPlayer: AnimationPlayer = null;
   constructor() {
     scheduleMicroTask(() => this._onFinish());
   }
+  /** @internal */
   _onFinish() {
     this._subscriptions.forEach(entry => { entry(); });
     this._subscriptions = [];

modules/@angular/core/testing/testing_internal.ts

@@ -212,6 +212,7 @@ export class SpyObject {
     return object;
   }
 
+  /** @internal */
   _createGuinnessCompatibleSpy(name): GuinessCompatibleSpy {
     var newSpy: GuinessCompatibleSpy = <any>jasmine.createSpy(name);
     newSpy.andCallFake = <any>newSpy.and.callFake;

modules/@angular/facade/src/async.ts

@@ -109,6 +109,7 @@ export class EventEmitter<T> extends Subject<T> {
   // we can't mark it as internal now because EventEmitter exported via @angular/core would not
   // contain this property making it incompatible with all the code that uses EventEmitter via
   // facades, which are local to the code and do not have this property stripped.
+  // tslint:disable-next-line
   __isAsync: boolean;
 
   /**

tslint.json

@@ -1,5 +1,6 @@
 {
     "rules": {
+      "requireInternalWithUnderscore": true,
       "duplicateModuleImport": true,
       "semicolon": true,
       "variable-name": false