docs: add changelog for 5.0.2

Using `display: none` on the `<h1>` causes `innerText` to not work as expected
and include the icon ligature (`link`) in the title. This caused the window
title on the angular.io Home page to appear as "Angular - link".
This commit fixes it by not generating anchors at all for headings with the
`no-anchor` class.

Fixes #20427

PR Close #20440

.bazelrc

@@ -0,0 +1,18 @@
+# Make compilation fast, by keeping a few copies of the compilers
+# running as daemons, and cache SourceFile AST's to reduce parse time.
+build --strategy=TypeScriptCompile=worker
+build --strategy=AngularTemplateCompile=worker
+
+# Don't create bazel-* symlinks in the WORKSPACE directory.
+# These require .gitignore and may scare users.
+# Also, it's a workaround for https://github.com/bazelbuild/rules_typescript/issues/12
+# which affects the common case of having `tsconfig.json` in the WORKSPACE directory.
+#
+# Instead, you should run `bazel info bazel-bin` to find out where the outputs went.
+build --symlink_prefix=/
+
+# Performance: avoid stat'ing input files
+build --watchfs
+
+# Don't print all the .d.ts output locations after builds
+build --show_result=0

.pullapprove.yml

@@ -22,6 +22,7 @@
 # petebacondarwin - Pete Bacon Darwin
 # pkozlowski-opensource - Pawel Kozlowski
 # robwormald - Rob Wormald
+# tbosch - Tobias Bosch
 # tinayuangao - Tina Gao
 # vicb - Victor Berchet
 # vikerman - Vikram Subramanian
@@ -99,6 +100,7 @@ groups:
     users:
       - alexeagle
       - mhevery
+      - tbosch
       - vicb
       - IgorMinar #fallback
 
@@ -107,7 +109,8 @@ groups:
       files:
         - "packages/core/*"
     users:
-      - chuckjaz #primary
+      - tbosch #primary
+      - chuckjaz
       - mhevery
       - vicb
       - IgorMinar #fallback
@@ -129,7 +132,7 @@ groups:
         - "packages/compiler/src/i18n/*"
     users:
       - vicb #primary
-      - chuckjaz
+      - tbosch
       - IgorMinar #fallback
       - mhevery #fallback
 
@@ -138,8 +141,9 @@ groups:
       files:
         - "packages/compiler/*"
     users:
-      - chuckjaz #primary
+      - tbosch #primary
       - vicb
+      - chuckjaz
       - mhevery
       - IgorMinar #fallback
 
@@ -159,11 +163,12 @@ groups:
           - "packages/compiler-cli/*"
           - "packages/bazel/*"
         exclude:
-          - "packages/compiler-cli/src/ngtools*"
+          - "packages/compiler-cli/src/ngtools*"   
     users:
       - alexeagle
       - chuckjaz
       - vicb
+      - tbosch
       - IgorMinar #fallback
       - mhevery #fallback
 
@@ -207,7 +212,7 @@ groups:
         - "packages/language-service/*"
     users:
       - chuckjaz #primary
-      # needs secondary
+      - tbosch #secondary
       - vicb
       - IgorMinar #fallback
       - mhevery #fallback
@@ -237,8 +242,8 @@ groups:
       files:
         - "packages/platform-browser/*"
     users:
-      - vicb #primary
-      # needs secondary
+      - tbosch #primary
+      - vicb #secondary
       - IgorMinar #fallback
       - mhevery #fallback
 
@@ -248,9 +253,9 @@ groups:
         - "packages/platform-server/*"
     users:
       - vikerman #primary
-      # needs secondary
       - alxhub
       - vicb
+      - tbosch
       - IgorMinar #fallback
       - mhevery #fallback
 
@@ -260,7 +265,7 @@ groups:
         - "packages/platform-webworker/*"
     users:
       - vicb #primary
-      # needs secondary
+      - tbosch #secondary
       - IgorMinar #fallback
       - mhevery #fallback
 
@@ -279,7 +284,7 @@ groups:
       files:
         - "packages/benchpress/*"
     users:
-      # needs primary
+      - tbosch #primary
       # needs secondary
       - IgorMinar #fallback
       - mhevery #fallback

CHANGELOG.md

@@ -1,142 +1,3 @@
-<a name="5.1.0-rc.1"></a>
-# [5.1.0-rc.1](https://github.com/angular/angular/compare/5.1.0-rc.0...5.1.0-rc.1) (2017-12-01)
-
-
-### Bug Fixes
-
-* **compiler-cli:** propagate ts.SourceFile moduleName into metadata ([f841fbe](https://github.com/angular/angular/commit/f841fbe))
-* **service-worker:** allow disabling SW while still using services ([65f4fad](https://github.com/angular/angular/commit/65f4fad))
-* **service-worker:** don't crash if SW not supported ([b9a91a5](https://github.com/angular/angular/commit/b9a91a5))
-* **service-worker:** send initialization signal from the application ([3fbcde9](https://github.com/angular/angular/commit/3fbcde9))
-* **service-worker:** use relative path for ngsw.json ([f582620](https://github.com/angular/angular/commit/f582620))
-
-
-
-<a name="5.0.5"></a>
-## [5.0.5](https://github.com/angular/angular/compare/5.0.4...5.0.5) (2017-12-01)
-
-
-### Bug Fixes
-
-* **compiler-cli:** propagate ts.SourceFile moduleName into metadata ([a2ff4ab](https://github.com/angular/angular/commit/a2ff4ab))
-* **service-worker:** allow disabling SW while still using services ([f99335b](https://github.com/angular/angular/commit/f99335b))
-* **service-worker:** don't crash if SW not supported ([ee37d4b](https://github.com/angular/angular/commit/ee37d4b))
-* **service-worker:** send initialization signal from the application ([6bf07b4](https://github.com/angular/angular/commit/6bf07b4))
-* **service-worker:** use relative path for ngsw.json ([56c98f7](https://github.com/angular/angular/commit/56c98f7))
-
-
-
-<a name="5.1.0-rc.0"></a>
-# [5.1.0-rc.0](https://github.com/angular/angular/compare/5.1.0-beta.2...5.1.0-rc.0) (2017-12-01)
-
-
-### Bug Fixes
-
-* **animations:** ensure multi-level enter animations work ([#19455](https://github.com/angular/angular/issues/19455)) ([dd6237e](https://github.com/angular/angular/commit/dd6237e))
-* **animations:** ensure multi-level enter animations work ([#19455](https://github.com/angular/angular/issues/19455)) ([b2a586c](https://github.com/angular/angular/commit/b2a586c))
-* **animations:** ensure multi-level leave animations work ([#19455](https://github.com/angular/angular/issues/19455)) ([1366762](https://github.com/angular/angular/commit/1366762))
-* **animations:** ensure multi-level leave animations work ([#19455](https://github.com/angular/angular/issues/19455)) ([c2b3792](https://github.com/angular/angular/commit/c2b3792))
-* **bazel:** produce named AMD modules for codegen ([#20547](https://github.com/angular/angular/issues/20547)) ([6e83204](https://github.com/angular/angular/commit/6e83204)), closes [#19422](https://github.com/angular/angular/issues/19422)
-* **common:** accept falsy values as HTTP bodies ([#19958](https://github.com/angular/angular/issues/19958)) ([15a54df](https://github.com/angular/angular/commit/15a54df)), closes [#19825](https://github.com/angular/angular/issues/19825) [#19195](https://github.com/angular/angular/issues/19195)
-* **common:** don't strip XSSI prefix for if error isn't JSON ([#19958](https://github.com/angular/angular/issues/19958)) ([aafa75d](https://github.com/angular/angular/commit/aafa75d))
-* **common:** remove useless guard in HttpClient ([#19958](https://github.com/angular/angular/issues/19958)) ([eb01ad5](https://github.com/angular/angular/commit/eb01ad5)), closes [#19223](https://github.com/angular/angular/issues/19223)
-* **common:** treat an empty body as null when parsing JSON in HttpClient ([#19958](https://github.com/angular/angular/issues/19958)) ([503be69](https://github.com/angular/angular/commit/503be69)), closes [#18680](https://github.com/angular/angular/issues/18680) [#19413](https://github.com/angular/angular/issues/19413) [#19502](https://github.com/angular/angular/issues/19502) [#19555](https://github.com/angular/angular/issues/19555)
-* **compiler:** correctly detect when to serialze summary metadata ([#20668](https://github.com/angular/angular/issues/20668)) ([8bb42df](https://github.com/angular/angular/commit/8bb42df))
-* **compiler-cli:** fix memory leak in program creation ([#20692](https://github.com/angular/angular/issues/20692)) ([71e5de6](https://github.com/angular/angular/commit/71e5de6)), closes [#20691](https://github.com/angular/angular/issues/20691)
-* **compiler-cli:** normalize sourcepaths for i18n extracted files ([#20417](https://github.com/angular/angular/issues/20417)) ([de78307](https://github.com/angular/angular/commit/de78307)), closes [#20416](https://github.com/angular/angular/issues/20416)
-* **core:** should use native addEventListener in ngZone ([#20672](https://github.com/angular/angular/issues/20672)) ([65a2cb8](https://github.com/angular/angular/commit/65a2cb8))
-* **language-service:** Allow empty templates ([#20651](https://github.com/angular/angular/issues/20651)) ([3203069](https://github.com/angular/angular/commit/3203069)), closes [#19406](https://github.com/angular/angular/issues/19406)
-* **language-service:** Fix crash when no script files are found ([#20550](https://github.com/angular/angular/issues/20550)) ([54bfe14](https://github.com/angular/angular/commit/54bfe14)), closes [#19325](https://github.com/angular/angular/issues/19325)
-
-
-### Features
-
-* **common:** add locale id parameter to `registerLocaleData` ([#20623](https://github.com/angular/angular/issues/20623)) ([24bf3e2](https://github.com/angular/angular/commit/24bf3e2))
-* **compiler-cli:** improve error messages produced during structural errors ([#20459](https://github.com/angular/angular/issues/20459)) ([8ecda94](https://github.com/angular/angular/commit/8ecda94))
-
-
-
-<a name="5.0.4"></a>
-## [5.0.4](https://github.com/angular/angular/compare/5.0.3...5.0.4) (2017-12-01)
-
-
-### Bug Fixes
-
-* **animations:** ensure multi-level enter animations work ([#19455](https://github.com/angular/angular/issues/19455)) ([22bbd6e](https://github.com/angular/angular/commit/22bbd6e))
-* **animations:** ensure multi-level leave animations work ([#19455](https://github.com/angular/angular/issues/19455)) ([c7b211c](https://github.com/angular/angular/commit/c7b211c))
-* **common:** accept falsy values as HTTP bodies ([#19958](https://github.com/angular/angular/issues/19958)) ([66fd1f8](https://github.com/angular/angular/commit/66fd1f8)), closes [#19825](https://github.com/angular/angular/issues/19825) [#19195](https://github.com/angular/angular/issues/19195)
-* **common:** don't strip XSSI prefix for if error isn't JSON ([#19958](https://github.com/angular/angular/issues/19958)) ([ead7596](https://github.com/angular/angular/commit/ead7596))
-* **common:** remove useless guard in HttpClient ([#19958](https://github.com/angular/angular/issues/19958)) ([e099911](https://github.com/angular/angular/commit/e099911)), closes [#19223](https://github.com/angular/angular/issues/19223)
-* **common:** treat an empty body as null when parsing JSON in HttpClient ([#19958](https://github.com/angular/angular/issues/19958)) ([bdaee50](https://github.com/angular/angular/commit/bdaee50)), closes [#18680](https://github.com/angular/angular/issues/18680) [#19413](https://github.com/angular/angular/issues/19413) [#19502](https://github.com/angular/angular/issues/19502) [#19555](https://github.com/angular/angular/issues/19555)
-* **compiler-cli:** fix memory leak in program creation ([#20692](https://github.com/angular/angular/issues/20692)) ([38be44d](https://github.com/angular/angular/commit/38be44d)), closes [#20691](https://github.com/angular/angular/issues/20691)
-* **compiler-cli:** normalize sourcepaths for i18n extracted files ([#20417](https://github.com/angular/angular/issues/20417)) ([2b0c896](https://github.com/angular/angular/commit/2b0c896)), closes [#20416](https://github.com/angular/angular/issues/20416)
-
-
-
-<a name="5.1.0-beta.2"></a>
-# [5.1.0-beta.2](https://github.com/angular/angular/compare/5.1.0-beta.1...5.1.0-beta.2) (2017-11-22)
-
-
-### Bug Fixes
-
-* **animations:** always fire inner trigger callbacks even if blocked by parent animations ([#19753](https://github.com/angular/angular/issues/19753)) ([0e012c9](https://github.com/angular/angular/commit/0e012c9)), closes [#19100](https://github.com/angular/angular/issues/19100)
-* **animations:** always fire start and done callbacks in order for noop animations ([#20570](https://github.com/angular/angular/issues/20570)) ([ffb6dbe](https://github.com/angular/angular/commit/ffb6dbe))
-* **animations:** validate against trigger() names that use @ symbols ([#20326](https://github.com/angular/angular/issues/20326)) ([1861e41](https://github.com/angular/angular/commit/1861e41))
-* **benchpress:** Allow ignoring navigationStart events in perflog metric. ([#20312](https://github.com/angular/angular/issues/20312)) ([717ac5a](https://github.com/angular/angular/commit/717ac5a))
-* **common:** return ISubscription from Location.subscribe() ([#20429](https://github.com/angular/angular/issues/20429)) ([437a044](https://github.com/angular/angular/commit/437a044)), closes [#20406](https://github.com/angular/angular/issues/20406)
-* **compiler:** emit correct type-check-blocks with TemplateRef's ([#20463](https://github.com/angular/angular/issues/20463)) ([68b53c0](https://github.com/angular/angular/commit/68b53c0))
-* **compiler:** support event bindings in `fullTemplateTypeCheck` ([#20490](https://github.com/angular/angular/issues/20490)) ([4ed0439](https://github.com/angular/angular/commit/4ed0439))
-* **core:** fix [#20532](https://github.com/angular/angular/issues/20532), should be able to cancel listener from mixed zone ([#20538](https://github.com/angular/angular/issues/20538)) ([a740e4f](https://github.com/angular/angular/commit/a740e4f))
-* **core:** should support event.stopImmediatePropagation ([#20469](https://github.com/angular/angular/issues/20469)) ([997336b](https://github.com/angular/angular/commit/997336b))
-* **forms:** updateOn should check if change occurred ([#20358](https://github.com/angular/angular/issues/20358)) ([69c53c3](https://github.com/angular/angular/commit/69c53c3)), closes [#20259](https://github.com/angular/angular/issues/20259)
-
-
-### Features
-
-* **platform-browser-dynamic:** export `JitCompilerFactory` ([#20478](https://github.com/angular/angular/issues/20478)) ([d7a727c](https://github.com/angular/angular/commit/d7a727c)), closes [#20125](https://github.com/angular/angular/issues/20125)
-
-
-
-<a name="5.0.3"></a>
-## [5.0.3](https://github.com/angular/angular/compare/5.0.2...5.0.3) (2017-11-22)
-
-
-### Bug Fixes
-
-* **animations:** always fire inner trigger callbacks even if blocked by parent animations ([#19753](https://github.com/angular/angular/issues/19753)) ([814f062](https://github.com/angular/angular/commit/814f062)), closes [#19100](https://github.com/angular/angular/issues/19100)
-* **animations:** validate against trigger() names that use @ symbols ([#20326](https://github.com/angular/angular/issues/20326)) ([15795d0](https://github.com/angular/angular/commit/15795d0))
-* **benchpress:** Allow ignoring navigationStart events in perflog metric. ([#20312](https://github.com/angular/angular/issues/20312)) ([9ca6ee9](https://github.com/angular/angular/commit/9ca6ee9))
-* **common:** return ISubscription from Location.subscribe() ([#20429](https://github.com/angular/angular/issues/20429)) ([bc904b1](https://github.com/angular/angular/commit/bc904b1)), closes [#20406](https://github.com/angular/angular/issues/20406)
-* **compiler:** emit correct type-check-blocks with TemplateRef's ([#20463](https://github.com/angular/angular/issues/20463)) ([81f1d42](https://github.com/angular/angular/commit/81f1d42))
-* **compiler:** support event bindings in `fullTemplateTypeCheck` ([#20490](https://github.com/angular/angular/issues/20490)) ([b53ead4](https://github.com/angular/angular/commit/b53ead4))
-* **core:** fix [#20532](https://github.com/angular/angular/issues/20532), should be able to cancel listener from mixed zone ([#20538](https://github.com/angular/angular/issues/20538)) ([0feba49](https://github.com/angular/angular/commit/0feba49))
-* **core:** should support event.stopImmediatePropagation ([#20469](https://github.com/angular/angular/issues/20469)) ([82aace6](https://github.com/angular/angular/commit/82aace6))
-* **forms:** updateOn should check if change occurred ([#20358](https://github.com/angular/angular/issues/20358)) ([f9f2c20](https://github.com/angular/angular/commit/f9f2c20)), closes [#20259](https://github.com/angular/angular/issues/20259)
-
-
-<a name="5.1.0-beta.1"></a>
-# [5.1.0-beta.1](https://github.com/angular/angular/compare/5.1.0-beta.0...5.1.0-beta.1) (2017-11-16)
-
-
-### Bug Fixes
-
-* **animations:** always fire inner trigger callbacks even if blocked by parent animations ([#19753](https://github.com/angular/angular/issues/19753)) ([d47b2a6](https://github.com/angular/angular/commit/d47b2a6)), closes [#19100](https://github.com/angular/angular/issues/19100)
-* **animations:** ensure final state() styles are applied within @.disabled animations ([#20267](https://github.com/angular/angular/issues/20267)) ([20aafff](https://github.com/angular/angular/commit/20aafff)), closes [#20266](https://github.com/angular/angular/issues/20266)
-* **bazel:** adjust mock of tsconfig for ng_module rule unit test ([#20175](https://github.com/angular/angular/issues/20175)) ([c2a24b4](https://github.com/angular/angular/commit/c2a24b4))
-* **compiler:** fix corner cases in shadow CSS ([c32f5fd](https://github.com/angular/angular/commit/c32f5fd))
-* **compiler:** recognize @NgModule with a redundant @Injectable ([#20320](https://github.com/angular/angular/issues/20320)) ([c33a576](https://github.com/angular/angular/commit/c33a576))
-* **compiler:** show explanatory text in template errors ([#20313](https://github.com/angular/angular/issues/20313)) ([3257fcd](https://github.com/angular/angular/commit/3257fcd))
-* **core:** ensure init lifecycle events are called ([#20258](https://github.com/angular/angular/issues/20258)) ([24cf8b3](https://github.com/angular/angular/commit/24cf8b3))
-* **language-service:** pass compilerOptions.paths to ReflectorHost ([#20222](https://github.com/angular/angular/issues/20222)) ([eb8013e](https://github.com/angular/angular/commit/eb8013e))
-* **router:** 'merge' queryParamHandling strategy should be able to remove query params ([#19733](https://github.com/angular/angular/issues/19733)) ([a622e19](https://github.com/angular/angular/commit/a622e19)), closes [#18463](https://github.com/angular/angular/issues/18463) [#17202](https://github.com/angular/angular/issues/17202)
-* Update test code to type-check under TS 2.5 ([#20175](https://github.com/angular/angular/issues/20175)) ([5ec1717](https://github.com/angular/angular/commit/5ec1717))
-
-### Features
-
-* **typescript:** support TypeScript 2.5 ([a9f3e2b](https://github.com/angular/angular/commit/a9f3e2b)), closes [#20175](https://github.com/angular/angular/issues/20175)
-
-> Note, if you do `Injector.get(Token)` where `Token` has static members, you'll run into https://github.com/Microsoft/TypeScript/issues/20102 where the returned type is `{}` rather than `Token`. Use `Injector.get<Token>(Token)` to work around.
-
 <a name="5.0.2"></a>
 ## [5.0.2](https://github.com/angular/angular/compare/5.0.1...5.0.2) (2017-11-16)
 
@@ -150,22 +11,6 @@
 * **router:** 'merge' queryParamHandling strategy should be able to remove query params ([#19733](https://github.com/angular/angular/issues/19733)) ([b732fb9](https://github.com/angular/angular/commit/b732fb9)), closes [#18463](https://github.com/angular/angular/issues/18463) [#17202](https://github.com/angular/angular/issues/17202)
 
 
-<a name="5.1.0-beta.0"></a>
-# [5.1.0-beta.0](https://github.com/angular/angular/compare/5.0.0-rc.4...5.1.0-beta.0) (2017-11-08)
-
-### Bug Fixes
-
-* **compiler:** don't overwrite missingTranslation's value in JIT ([#19952](https://github.com/angular/angular/issues/19952)) ([799cbb9](https://github.com/angular/angular/commit/799cbb9))
-* **compiler:** report a reasonable error with invalid metadata ([#20062](https://github.com/angular/angular/issues/20062)) ([da22c48](https://github.com/angular/angular/commit/da22c48))
-* **compiler-cli:** don't report emit diagnostics when `--noEmitOnError` is off ([#20063](https://github.com/angular/angular/issues/20063)) ([8639995](https://github.com/angular/angular/commit/8639995))
-* **core:** `__symbol__` should return `__zone_symbol__` without zone.js loaded ([#19541](https://github.com/angular/angular/issues/19541)) ([678d1cf](https://github.com/angular/angular/commit/678d1cf))
-* **core:** should support event.stopImmediatePropagation ([#19222](https://github.com/angular/angular/issues/19222)) ([7083791](https://github.com/angular/angular/commit/7083791))
-* **platform-browser:** support Symbols in custom `jasmineToString()` method ([#19794](https://github.com/angular/angular/issues/19794)) ([5a6efa7](https://github.com/angular/angular/commit/5a6efa7))
-
-### Features
-
-* **compiler:** introduce `TestBed.overrideTemplateUsingTestingModule` ([a460066](https://github.com/angular/angular/commit/a460066)), closes [#19815](https://github.com/angular/angular/issues/19815)
-
 
 <a name="5.0.1"></a>
 ## [5.0.1](https://github.com/angular/angular/compare/5.0.0...5.0.1) (2017-11-08)

CONTRIBUTING.md

@@ -69,37 +69,36 @@ You can file new issues by filling out our [new issue form](https://github.com/a
 ### <a name="submit-pr"></a> Submitting a Pull Request (PR)
 Before you submit your Pull Request (PR) consider the following guidelines:
 
-1. Search [GitHub](https://github.com/angular/angular/pulls) for an open or closed PR
+* Search [GitHub](https://github.com/angular/angular/pulls) for an open or closed PR
   that relates to your submission. You don't want to duplicate effort.
-1. Please sign our [Contributor License Agreement (CLA)](#cla) before sending PRs.
+* Please sign our [Contributor License Agreement (CLA)](#cla) before sending PRs.
   We cannot accept code without this.
-1. Fork the angular/angular repo.
-1. Make your changes in a new git branch:
+* Make your changes in a new git branch:
 
      ```shell
      git checkout -b my-fix-branch master
      ```
 
-1. Create your patch, **including appropriate test cases**.
-1. Follow our [Coding Rules](#rules).
-1. Run the full Angular test suite, as described in the [developer documentation][dev-doc],
+* Create your patch, **including appropriate test cases**.
+* Follow our [Coding Rules](#rules).
+* Run the full Angular test suite, as described in the [developer documentation][dev-doc],
   and ensure that all tests pass.
-1. Commit your changes using a descriptive commit message that follows our
+* Commit your changes using a descriptive commit message that follows our
   [commit message conventions](#commit). Adherence to these conventions
   is necessary because release notes are automatically generated from these messages.
 
      ```shell
      git commit -a
      ```
-    Note: the optional commit `-a` command line option will automatically "add" and "rm" edited files.
+  Note: the optional commit `-a` command line option will automatically "add" and "rm" edited files.
 
-1. Push your branch to GitHub:
+* Push your branch to GitHub:
 
     ```shell
     git push origin my-fix-branch
     ```
 
-1. In GitHub, send a pull request to `angular:master`.
+* In GitHub, send a pull request to `angular:master`.
 * If we suggest changes then:
   * Make the required updates.
   * Re-run the Angular test suites to ensure tests are still passing.

WORKSPACE

@@ -5,7 +5,8 @@ load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository")
 git_repository(
     name = "build_bazel_rules_nodejs",
     remote = "https://github.com/bazelbuild/rules_nodejs.git",
-    commit = "0.2.1",
+    # TODO(alexeagle): use the correct tag here.
+    commit = "2c6243df53fd33fdab283ebdd01582e4eb815db8",
 )
 
 load("@build_bazel_rules_nodejs//:defs.bzl", "node_repositories")

aio/content/examples/attribute-directives/src/app/app.component.1.html

@@ -1,14 +1,14 @@
 <!-- #docregion -->
 <h1>My First Attribute Directive</h1>
 <!-- #docregion applied -->
-<p appHighlight>Highlight me!</p>
+<p appHightlight>Highlight me!</p>
 <!-- #enddocregion applied,  -->
 
 <!-- #docregion color-1 -->
-<p appHighlight highlightColor="yellow">Highlighted in yellow</p>
-<p appHighlight [highlightColor]="'orange'">Highlighted in orange</p>
+<p appHightlight highlightColor="yellow">Highlighted in yellow</p>
+<p appHightlight [highlightColor]="'orange'">Highlighted in orange</p>
 <!-- #enddocregion color-1 -->
 
 <!-- #docregion color-2 -->
-<p appHighlight [highlightColor]="color">Highlighted with parent component's color</p>
+<p appHightlight [highlightColor]="color">Highlighted with parent component's color</p>
 <!-- #enddocregion color-2 -->

aio/content/examples/lifecycle-hooks/src/app/after-content.component.ts

@@ -93,20 +93,22 @@ export class AfterContentComponent implements AfterContentChecked, AfterContentI
 
     <h4>-- AfterContent Logs --</h4>
     <p><button (click)="reset()">Reset</button></p>
-    <div *ngFor="let msg of logger.logs">{{msg}}</div>
+    <div *ngFor="let msg of logs">{{msg}}</div>
   </div>
   `,
   styles: ['.parent {background: burlywood}'],
   providers: [LoggerService]
 })
 export class AfterContentParentComponent {
+  logs: string[];
   show = true;
 
-  constructor(public logger: LoggerService) {
+  constructor(private logger: LoggerService) {
+    this.logs = logger.logs;
   }
 
   reset() {
-    this.logger.clear();
+    this.logs.length = 0;
     // quickly remove and reload AfterContentComponent which recreates it
     this.show = false;
     this.logger.tick_then(() => this.show = true);

aio/content/examples/lifecycle-hooks/src/app/after-view.component.ts

@@ -95,20 +95,22 @@ export class AfterViewComponent implements  AfterViewChecked, AfterViewInit {
 
     <h4>-- AfterView Logs --</h4>
     <p><button (click)="reset()">Reset</button></p>
-    <div *ngFor="let msg of logger.logs">{{msg}}</div>
+    <div *ngFor="let msg of logs">{{msg}}</div>
   </div>
   `,
   styles: ['.parent {background: burlywood}'],
   providers: [LoggerService]
 })
 export class AfterViewParentComponent {
+  logs: string[];
   show = true;
 
-  constructor(public logger: LoggerService) {
+  constructor(private logger: LoggerService) {
+    this.logs = logger.logs;
   }
 
   reset() {
-    this.logger.clear();
+    this.logs.length = 0;
     // quickly remove and reload AfterViewComponent which recreates it
     this.show = false;
     this.logger.tick_then(() => this.show = true);

aio/content/examples/lifecycle-hooks/src/app/counter.component.ts

@@ -27,7 +27,7 @@ export class MyCounterComponent implements OnChanges {
     // Empty the changeLog whenever counter goes to zero
     // hint: this is a way to respond programmatically to external value changes.
     if (this.counter === 0) {
-      this.changeLog = [];
+      this.changeLog.length = 0;
     }
 
     // A change to `counter` is the only change we care about

aio/content/examples/lifecycle-hooks/src/app/do-check.component.ts

@@ -68,7 +68,7 @@ export class DoCheckComponent implements DoCheck {
 
   reset() {
     this.changeDetected = true;
-    this.changeLog = [];
+    this.changeLog.length = 0;
   }
 }
 

aio/content/examples/lifecycle-hooks/src/app/logger.service.ts

@@ -18,7 +18,7 @@ export class LoggerService {
     }
   }
 
-  clear() { this.logs = []; }
+  clear() { this.logs.length = 0; }
 
   // schedules a view refresh to ensure display catches up
   tick() {  this.tick_then(() => { }); }

aio/content/examples/lifecycle-hooks/src/app/on-changes.component.ts

@@ -43,7 +43,7 @@ export class OnChangesComponent implements OnChanges {
   }
   // #enddocregion ng-on-changes
 
-  reset() { this.changeLog = []; }
+  reset() { this.changeLog.length = 0; }
 }
 
 /***************************************/

aio/content/examples/lifecycle-hooks/src/app/spy.component.html

@@ -12,5 +12,5 @@
   </div>
   <!-- #enddocregion template -->
   <h4>-- Spy Lifecycle Hook Log --</h4>
-  <div *ngFor="let msg of logger.logs">{{msg}}</div>
+  <div *ngFor="let msg of spyLog">{{msg}}</div>
 </div>

aio/content/examples/lifecycle-hooks/src/app/spy.component.ts

@@ -15,8 +15,10 @@ import { LoggerService }  from './logger.service';
 export class SpyParentComponent {
   newName = 'Herbie';
   heroes: string[] = ['Windstorm', 'Magneta'];
+  spyLog: string[];
 
-  constructor(public logger: LoggerService) {
+  constructor(private logger: LoggerService) {
+    this.spyLog = logger.logs;
   }
 
   addHero() {
@@ -32,7 +34,7 @@ export class SpyParentComponent {
   }
   reset() {
     this.logger.log('-- reset --');
-    this.heroes = [];
+    this.heroes.length = 0;
     this.logger.tick();
   }
 }

aio/content/examples/ngmodule/contact.1b.plnkr.json

@@ -14,7 +14,7 @@
     "app/contact/contact.component.html",
     "app/contact/contact.component.3.ts",
     "app/contact/contact.service.ts",
-    "app/contact/contact-highlight.directive.ts",
+    "app/contact/highlight.directive.ts",
 
     "main.1b.ts",
     "styles.css",

aio/content/examples/ngmodule/contact.2.plnkr.json

@@ -16,7 +16,7 @@
     "app/contact/awesome.pipe.ts",
     "app/contact/contact.component.3.ts",
     "app/contact/contact.module.2.ts",
-    "app/contact/contact-highlight.directive.ts",
+    "app/contact/highlight.directive.ts",
 
     "main.2.ts",
     "styles.css",

aio/content/examples/ngmodule/e2e/app.e2e-spec.ts

@@ -15,7 +15,7 @@ describe('NgModule', function () {
 
     return {
       title: element.all(by.tagName('h1')).get(0),
-      welcome: element.all(by.css('app-title p i')).get(0),
+      subtitle: element.all(by.css('app-title p i')).get(0),
       contactButton: buttons.get(0),
       crisisButton: buttons.get(1),
       heroesButton: buttons.get(2)
@@ -67,7 +67,7 @@ describe('NgModule', function () {
 
       it('should welcome us', function () {
         const commons = getCommonsSectionStruct();
-        expect(commons.welcome.getText()).toBe('Welcome, ' + (name ||  'Sherlock Holmes'));
+        expect(commons.subtitle.getText()).toBe('Welcome, ' + (name ||  'Sherlock Holmes'));
       });
     };
   }

aio/content/examples/ngmodule/pre-shared.3.plnkr.json

@@ -19,7 +19,7 @@
     "app/contact/contact.component.3.ts",
     "app/contact/contact.module.3.ts",
     "app/contact/contact-routing.module.3.ts",
-    "app/contact/contact-highlight.directive.ts",
+    "app/contact/highlight.directive.ts",
 
     "app/crisis/*.ts",
 

aio/content/examples/ngmodule/src/app/app-routing.module.3.ts

@@ -1,19 +1,14 @@
 import { NgModule } from '@angular/core';
 import { Routes, RouterModule } from '@angular/router';
 
-import { ContactModule } from './contact/contact.module.3';
-
-const routes: Routes = [
+export const routes: Routes = [
   { path: '', redirectTo: 'contact', pathMatch: 'full'},
   { path: 'crisis', loadChildren: 'app/crisis/crisis.module#CrisisModule' },
   { path: 'heroes', loadChildren: 'app/hero/hero.module.3#HeroModule' }
 ];
 
 @NgModule({
-  imports: [
-    ContactModule,
-    RouterModule.forRoot(routes)
-  ],
+  imports: [RouterModule.forRoot(routes)],
   exports: [RouterModule]
 })
 export class AppRoutingModule {}

aio/content/examples/ngmodule/src/app/app-routing.module.ts

@@ -2,29 +2,18 @@
 import { NgModule }             from '@angular/core';
 import { Routes, RouterModule } from '@angular/router';
 
-import { ContactModule }    from './contact/contact.module';
-
-// #docregion routes
-const routes: Routes = [
+export const routes: Routes = [
   { path: '', redirectTo: 'contact', pathMatch: 'full'},
-  // #docregion lazy-routes
+// #docregion lazy-routes
   { path: 'crisis', loadChildren: 'app/crisis/crisis.module#CrisisModule' },
   { path: 'heroes', loadChildren: 'app/hero/hero.module#HeroModule' }
-  // #enddocregion lazy-routes
+// #enddocregion lazy-routes
 ];
-// #enddocregion routes
 
+// #docregion forRoot
 @NgModule({
-  // #docregion imports
-  imports: [
-    ContactModule,
-    // #docregion forRoot
-    RouterModule.forRoot(routes),
-    // #enddocregion forRoot
-  ],
-  // #enddocregion imports
-  // #docregion exports
+  imports: [RouterModule.forRoot(routes)],
   exports: [RouterModule]
-  // #enddocregion exports
 })
 export class AppRoutingModule {}
+// #enddocregion forRoot

aio/content/examples/ngmodule/src/app/app.component.0.ts

@@ -6,5 +6,5 @@ import { Component } from '@angular/core';
   template: '<h1>{{title}}</h1>',
 })
 export class AppComponent {
-  title = 'Angular Modules';
+  title = 'Minimal NgModule';
 }

aio/content/examples/ngmodule/src/app/app.component.1.ts

@@ -11,7 +11,9 @@ import { Component } from '@angular/core';
   // #enddocregion template
   */
 // #docregion
-  template: '<app-title></app-title>'
+  template: '<app-title [subtitle]="subtitle"></app-title>'
 })
-export class AppComponent {}
+export class AppComponent {
+  subtitle = '(v1)';
+}
 // #enddocregion

aio/content/examples/ngmodule/src/app/app.component.1b.ts

@@ -5,9 +5,11 @@ import { Component } from '@angular/core';
   selector: 'app-root',
   // #docregion template
   template: `
-    <app-title></app-title>
+    <app-title [subtitle]="subtitle"></app-title>
     <app-contact></app-contact>
   `
   // #enddocregion template
 })
-export class AppComponent {}
+export class AppComponent {
+  subtitle = '(v1)';
+}

aio/content/examples/ngmodule/src/app/app.component.2.ts

@@ -3,8 +3,10 @@ import { Component } from '@angular/core';
 @Component({
   selector: 'app-root',
   template: `
-    <app-title></app-title>
+    <app-title [subtitle]="subtitle"></app-title>
     <app-contact></app-contact>
   `
 })
-export class AppComponent {}
+export class AppComponent {
+  subtitle = '(v2)';
+}

aio/content/examples/ngmodule/src/app/app.component.3.ts

@@ -4,7 +4,7 @@ import { Component } from '@angular/core';
   selector: 'app-root',
   // #docregion template
   template: `
-    <app-title></app-title>
+    <app-title [subtitle]="subtitle"></app-title>
     <nav>
       <a routerLink="contact" routerLinkActive="active">Contact</a>
       <a routerLink="crisis"  routerLinkActive="active">Crisis Center</a>
@@ -14,4 +14,6 @@ import { Component } from '@angular/core';
   `
   // #enddocregion template
 })
-export class AppComponent {}
+export class AppComponent {
+  subtitle = '(v3)';
+}

aio/content/examples/ngmodule/src/app/app.component.ts

@@ -5,7 +5,7 @@ import { Component } from '@angular/core';
 @Component({
   selector: 'app-root',
   template: `
-    <app-title></app-title>
+    <app-title [subtitle]="subtitle"></app-title>
     <nav>
       <a routerLink="contact" routerLinkActive="active">Contact</a>
       <a routerLink="crisis"  routerLinkActive="active">Crisis Center</a>
@@ -14,4 +14,6 @@ import { Component } from '@angular/core';
     <router-outlet></router-outlet>
   `
 })
-export class AppComponent {}
+export class AppComponent {
+  subtitle = '(Final)';
+}

aio/content/examples/ngmodule/src/app/app.module.0.ts

@@ -1,7 +1,17 @@
+// #docplaster
+// #docregion
 import { NgModule }      from '@angular/core';
 import { BrowserModule } from '@angular/platform-browser';
 
-import { AppComponent }  from './app.component.0';
+import
+// #enddocregion
+       { AppComponent }  from './app.component.0';
+/*
+// #docregion
+       { AppComponent }  from './app.component';
+// #enddocregion
+*/
+// #docregion
 
 @NgModule({
 // #docregion imports

aio/content/examples/ngmodule/src/app/app.module.1.ts

@@ -1,15 +1,14 @@
 // #docplaster
 // #docregion
-/* Angular Imports */
 import { NgModule }           from '@angular/core';
 import { BrowserModule }      from '@angular/platform-browser';
 
-/* App Imports */
+import
 // #enddocregion
-import { AppComponent }       from './app.component.1';
+       { AppComponent }       from './app.component.1';
 /*
 // #docregion
-import { AppComponent }       from './app.component';
+       { AppComponent }       from './app.component';
 // #enddocregion
 */
 // #docregion
@@ -22,9 +21,12 @@ import { FormsModule }        from '@angular/forms';
 
 import { AwesomePipe }        from './contact/awesome.pipe';
 import { ContactComponent }   from './contact/contact.component.3';
+
+// #docregion import-contact-directive
 import {
-  ContactHighlightDirective as ContactHighlightDirective
-} from './contact/contact-highlight.directive';
+  HighlightDirective as ContactHighlightDirective
+} from './contact/highlight.directive';
+// #enddocregion import-contact-directive
 
 @NgModule({
 // #docregion imports

aio/content/examples/ngmodule/src/app/app.module.1b.ts

@@ -1,16 +1,15 @@
 // #docplaster
 // #docregion
-/* Angular Imports */
 import { NgModule }           from '@angular/core';
 import { BrowserModule }      from '@angular/platform-browser';
-import { FormsModule }        from '@angular/forms';
 
-/* App Imports */
+/* App Root */
+import
 // #enddocregion
-import { AppComponent }       from './app.component.1b';
+       { AppComponent }       from './app.component.1b';
 /*
 // #docregion
-import { AppComponent }       from './app.component';
+       { AppComponent }       from './app.component';
 // #enddocregion
 */
 // #docregion
@@ -19,17 +18,25 @@ import { TitleComponent }     from './title.component';
 import { UserService }        from './user.service';
 
 /* Contact Imports */
+import
 // #enddocregion
-import { ContactComponent } from './contact/contact.component.3';
+       { ContactComponent }   from './contact/contact.component.3';
 /*
 // #docregion
-import { ContactComponent } from './contact/contact.component';
+       { ContactComponent }   from './contact/contact.component';
 // #enddocregion
 */
 // #docregion
-import { AwesomePipe } from './contact/awesome.pipe';
-import { ContactService } from './contact/contact.service';
-import { ContactHighlightDirective } from './contact/contact-highlight.directive';
+import { ContactService }     from './contact/contact.service';
+import { AwesomePipe }        from './contact/awesome.pipe';
+
+// #docregion import-alias
+import {
+  HighlightDirective as ContactHighlightDirective
+} from './contact/highlight.directive';
+// #enddocregion import-alias
+
+import { FormsModule }        from '@angular/forms';
 
 @NgModule({
   imports: [ BrowserModule,  FormsModule ],

aio/content/examples/ngmodule/src/app/app.module.2.ts

@@ -1,15 +1,15 @@
 // #docplaster
 // #docregion
-/* Angular Imports */
 import { NgModule }           from '@angular/core';
 import { BrowserModule }      from '@angular/platform-browser';
 
-/* App Imports */
+/* App Root */
+import
 // #enddocregion
-import { AppComponent }       from './app.component.2';
+       { AppComponent }       from './app.component.2';
 /*
 // #docregion
-import { AppComponent }       from './app.component';
+       { AppComponent }       from './app.component';
 // #enddocregion
 */
 // #docregion
@@ -18,11 +18,12 @@ import { TitleComponent }     from './title.component';
 import { UserService }        from './user.service';
 
 /* Contact Imports */
+import
 // #enddocregion
-import { ContactModule }      from './contact/contact.module.2';
+       { ContactModule }      from './contact/contact.module.2';
 /*
 // #docregion
-import { ContactModule }      from './contact/contact.module';
+       { ContactModule }      from './contact/contact.module';
 // #enddocregion
 */
 // #docregion

aio/content/examples/ngmodule/src/app/app.module.3.ts

@@ -1,36 +1,25 @@
 // #docplaster
 // #docregion
-/* Angular Imports */
 import { NgModule }           from '@angular/core';
 import { BrowserModule }      from '@angular/platform-browser';
 
-/* App Imports */
-// #enddocregion
+/* App Root */
 import { AppComponent }       from './app.component.3';
-/*
-// #docregion
-import { AppComponent }       from './app.component';
-// #enddocregion
-*/
-// #docregion
 import { HighlightDirective } from './highlight.directive';
 import { TitleComponent }     from './title.component';
 import { UserService }        from './user.service';
 
+/* Feature Modules */
+import { ContactModule }      from './contact/contact.module.3';
+
 /* Routing Module */
-// #enddocregion
 import { AppRoutingModule }   from './app-routing.module.3';
-/*
-// #docregion
-import { AppRoutingModule }   from './app-routing.module';
-// #enddocregion
-*/
-// #docregion
 
 @NgModule({
 // #docregion imports
   imports:      [
     BrowserModule,
+    ContactModule,
     AppRoutingModule
   ],
 // #enddocregion imports

aio/content/examples/ngmodule/src/app/app.module.ts

@@ -1,14 +1,14 @@
 // #docplaster
 // #docregion
 // #docregion v4
-/* Angular Imports */
 import { NgModule }       from '@angular/core';
 import { BrowserModule }  from '@angular/platform-browser';
 
-/* App Imports */
+/* App Root */
 import { AppComponent }   from './app.component';
 
-/* Core Modules */
+/* Feature Modules */
+import { ContactModule }    from './contact/contact.module';
 import { CoreModule }       from './core/core.module';
 
 /* Routing Module */
@@ -18,6 +18,7 @@ import { AppRoutingModule } from './app-routing.module';
   // #docregion import-for-root
   imports: [
     BrowserModule,
+    ContactModule,
 // #enddocregion v4
 // #enddocregion import-for-root
 /*

aio/content/examples/ngmodule/src/app/contact/contact-routing.module.3.ts

@@ -3,12 +3,10 @@ import { RouterModule }        from '@angular/router';
 
 import { ContactComponent }    from './contact.component.3';
 
-const routes = [
-  { path: 'contact', component: ContactComponent}
-];
-
 @NgModule({
-  imports: [ RouterModule.forChild(routes) ],
-  exports: [ RouterModule ]
+  imports: [RouterModule.forChild([
+    { path: 'contact', component: ContactComponent}
+  ])],
+  exports: [RouterModule]
 })
 export class ContactRoutingModule {}

aio/content/examples/ngmodule/src/app/contact/contact-routing.module.ts

@@ -4,13 +4,11 @@ import { RouterModule }        from '@angular/router';
 import { ContactComponent }    from './contact.component';
 
 // #docregion routing
-const routes = [
-  { path: 'contact', component: ContactComponent}
-];
-
 @NgModule({
-  imports: [ RouterModule.forChild(routes) ],
-  exports: [ RouterModule ]
+  imports: [RouterModule.forChild([
+    { path: 'contact', component: ContactComponent }
+  ])],
+  exports: [RouterModule]
 })
 export class ContactRoutingModule {}
 // #enddocregion

aio/content/examples/ngmodule/src/app/contact/contact.component.3.ts

@@ -21,7 +21,7 @@ export class ContactComponent implements OnInit {
   }
 
   ngOnInit() {
-    this.contactService.getContacts().subscribe(contacts => {
+    this.contactService.getContacts().then(contacts => {
       this.msg = '';
       this.contacts = contacts;
       this.contact = contacts[0];

aio/content/examples/ngmodule/src/app/contact/contact.component.css

@@ -27,6 +27,3 @@
   margin-bottom: 20px;
 }
 
-.button-group {
-  padding-top: 12px;
-}

aio/content/examples/ngmodule/src/app/contact/contact.component.html

@@ -6,32 +6,18 @@
   <!-- #docregion awesome -->
   <h3 highlight>{{ contact.name | awesome }}</h3>
   <!-- #enddocregion awesome -->
-
   <div class="form-group">
     <label for="name">Name</label>
-
-    <!-- #docregion ngModel -->
     <input type="text" class="form-control" required
       [(ngModel)]="contact.name"
-      name="name"  #name="ngModel" >
-    <!-- #enddocregion ngModel -->
-
+        name="name"  #name="ngModel" >
     <div [hidden]="name.valid" class="alert alert-danger">
       Name is required
     </div>
   </div>
-
-  <div class="button-group">
-    <button type="submit" class="btn btn-default"
-    [disabled]="!contactForm.form.valid">
-    Save</button>
-
-    <button type="button" class="btn" (click)="next()"
-    [disabled]="!contactForm.form.valid">
-    Next Contact</button>
-
-    <button type="button" class="btn" (click)="newContact()">
-    New Contact</button>
-  </div>
+  <br>
+  <button type="submit" class="btn btn-default" [disabled]="!contactForm.form.valid">Save</button>
+  <button type="button" class="btn" (click)="next()" [disabled]="!contactForm.form.valid">Next Contact</button>
+  <button type="button" class="btn" (click)="newContact()">New Contact</button>
 </form>
 <!-- #enddocregion -->

aio/content/examples/ngmodule/src/app/contact/contact.component.ts

@@ -22,7 +22,7 @@ export class ContactComponent implements OnInit {
   }
 
   ngOnInit() {
-    this.contactService.getContacts().subscribe(contacts => {
+    this.contactService.getContacts().then(contacts => {
       this.msg = '';
       this.contacts = contacts;
       this.contact = contacts[0];

aio/content/examples/ngmodule/src/app/contact/contact.module.0.ts

@@ -1,11 +0,0 @@
-// #docregion
-import { NgModule } from '@angular/core';
-import { CommonModule } from '@angular/common';
-
-@NgModule({
-  imports: [
-    CommonModule
-  ],
-  declarations: []
-})
-export class ContactModule { }

aio/content/examples/ngmodule/src/app/contact/contact.module.2.ts

@@ -5,32 +5,25 @@ import { CommonModule }       from '@angular/common';
 import { FormsModule }        from '@angular/forms';
 
 import { AwesomePipe }        from './awesome.pipe';
+
+import
 // #enddocregion
-import { ContactComponent }   from './contact.component.3';
+       { ContactComponent }   from './contact.component.3';
 /*
 // #docregion
-import { ContactComponent }   from './contact.component';
+       { ContactComponent }   from './contact.component';
 // #enddocregion
 */
 // #docregion
-import { ContactHighlightDirective } from './contact-highlight.directive';
 import { ContactService }     from './contact.service';
+import { HighlightDirective } from './highlight.directive';
 
 // #docregion class
 @NgModule({
-  imports: [
-    CommonModule,
-    FormsModule
-  ],
-  declarations: [
-    AwesomePipe,
-    ContactComponent,
-    ContactHighlightDirective
-  ],
-  // #docregion exports
-  exports:   [ ContactComponent ],
-  // #enddocregion exports
-  providers: [ ContactService ]
+  imports:      [ CommonModule, FormsModule ],
+  declarations: [ ContactComponent, HighlightDirective, AwesomePipe ],
+  exports:      [ ContactComponent ],
+  providers:    [ ContactService ]
 })
 export class ContactModule { }
 // #enddocregion class

aio/content/examples/ngmodule/src/app/contact/contact.module.3.ts

@@ -1,43 +1,21 @@
-// #docplaster
 // #docregion
 import { NgModule }           from '@angular/core';
 import { CommonModule }       from '@angular/common';
 import { FormsModule }        from '@angular/forms';
 
 import { AwesomePipe }        from './awesome.pipe';
-// #enddocregion
-import { ContactComponent }   from './contact.component.3';
-/*
-// #docregion
-import { ContactComponent }   from './contact.component';
-// #enddocregion
-*/
-// #docregion
-import { ContactHighlightDirective } from './contact-highlight.directive';
-import { ContactService }     from './contact.service';
 
-// #enddocregion
+import { ContactComponent }   from './contact.component.3';
+import { ContactService }     from './contact.service';
+import { HighlightDirective } from './highlight.directive';
+
 import { ContactRoutingModule }   from './contact-routing.module.3';
-/*
-// #docregion
-import { ContactRoutingModule }   from './contact-routing.module';
-// #enddocregion
-*/
-// #docregion
 
 // #docregion class
 @NgModule({
-  imports: [
-    CommonModule,
-    FormsModule,
-    ContactRoutingModule
-  ],
-  declarations: [
-    AwesomePipe,
-    ContactComponent,
-    ContactHighlightDirective
-  ],
-  providers: [ ContactService ]
+  imports:      [ CommonModule, FormsModule, ContactRoutingModule ],
+  declarations: [ ContactComponent, HighlightDirective, AwesomePipe ],
+  providers:    [ ContactService ]
 })
 export class ContactModule { }
 // #enddocregion class

aio/content/examples/ngmodule/src/app/contact/contact.module.ts

@@ -8,10 +8,7 @@ import { ContactRoutingModule } from './contact-routing.module';
 
 // #docregion class
 @NgModule({
-  imports: [
-    SharedModule,
-    ContactRoutingModule
-  ],
+  imports:      [ SharedModule, ContactRoutingModule ],
   declarations: [ ContactComponent ],
   providers:    [ ContactService ]
 })

aio/content/examples/ngmodule/src/app/contact/contact.service.ts

@@ -1,10 +1,5 @@
-// #docplaster
 // #docregion
-import { Injectable, OnDestroy } from '@angular/core';
-
-import { Observable } from 'rxjs/Observable';
-import { of }         from 'rxjs/observable/of';
-import { delay }      from 'rxjs/operators';
+import { Injectable } from '@angular/core';
 
 export class Contact {
   constructor(public id: number, public name: string) { }
@@ -18,21 +13,17 @@ const CONTACTS: Contact[] = [
 
 const FETCH_LATENCY = 500;
 
-/** Simulate a data service that retrieves contacts from a server */
 @Injectable()
-export class ContactService implements OnDestroy {
-// #enddocregion
-  constructor() { console.log('ContactService instance created.'); }
-  ngOnDestroy() { console.log('ContactService instance destroyed.'); }
+export class ContactService {
 
-// #docregion
-  getContacts(): Observable<Contact[]> {
-    return of(CONTACTS).pipe(delay(FETCH_LATENCY));
+  getContacts() {
+    return new Promise<Contact[]>(resolve => {
+      setTimeout(() => { resolve(CONTACTS); }, FETCH_LATENCY);
+    });
   }
 
-  getContact(id: number | string): Observable<Contact> {
-    return of(CONTACTS.find(contact => contact.id === +id))
-      .pipe(delay(FETCH_LATENCY));
+  getContact(id: number | string) {
+    return this.getContacts()
+      .then(heroes => heroes.find(hero => hero.id === +id));
   }
 }
-// #enddocregion

aio/content/examples/ngmodule/src/app/contact/highlight.directive.ts

@@ -1,4 +1,4 @@
-// #docplaster
+/* tslint:disable */
 // Same directive name and selector as
 // HighlightDirective in parent AppModule
 // It selects for both input boxes and  'highlight' attr
@@ -7,14 +7,12 @@
 // #docregion
 import { Directive, ElementRef } from '@angular/core';
 
-// Highlight the host element or any InputElement in blue
 @Directive({ selector: '[highlight], input' })
-export class ContactHighlightDirective {
+/** Highlight the attached element or an InputElement in blue */
+export class HighlightDirective {
   constructor(el: ElementRef) {
     el.nativeElement.style.backgroundColor = 'powderblue';
-    // #enddocregion
-    console.log(`* Contact highlight called for ${el.nativeElement.tagName}`);
-    // #docregion
+    console.log(
+      `* Contact highlight called for ${el.nativeElement.tagName}`);
   }
 }
-// #enddocregion

aio/content/examples/ngmodule/src/app/core/title.component.html

@@ -1,5 +1,5 @@
 <!-- Exact copy from earlier app.component.html -->
-<h1 highlight>{{title}}</h1>
+<h1 highlight>{{title}} {{subtitle}}</h1>
 <p *ngIf="user">
   <i>Welcome, {{user}}</i>
 <p>

aio/content/examples/ngmodule/src/app/core/title.component.ts

@@ -7,6 +7,7 @@ import { UserService }      from '../core/user.service';
   templateUrl: './title.component.html',
 })
 export class TitleComponent {
+  @Input() subtitle = '';
   title = 'Angular Modules';
   user = '';
 

aio/content/examples/ngmodule/src/app/crisis/crisis-list.component.ts

@@ -1,21 +1,22 @@
-import { Component } from '@angular/core';
-import { Observable } from 'rxjs/Observable';
+import { Component, OnInit } from '@angular/core';
 
 import { Crisis,
-         CrisisService } from './crisis.service';
+         CrisisService }     from './crisis.service';
 
 @Component({
   template: `
     <h3 highlight>Crisis List</h3>
-    <div *ngFor='let crisis of crises | async'>
+    <div *ngFor='let crisis of crisises | async'>
       <a routerLink="{{'../' + crisis.id}}">{{crisis.id}} - {{crisis.name}}</a>
     </div>
   `
 })
-export class CrisisListComponent {
-  crises: Observable<Crisis[]>;
+export class CrisisListComponent implements OnInit {
+  crisises: Promise<Crisis[]>;
 
-  constructor(private crisisService: CrisisService) {
-    this.crises = this.crisisService.getCrises();
+  constructor(private crisisService: CrisisService) { }
+
+  ngOnInit() {
+    this.crisises = this.crisisService.getCrises();
   }
 }

aio/content/examples/ngmodule/src/app/crisis/crisis.service.ts

@@ -1,8 +1,4 @@
-import { Injectable, OnDestroy } from '@angular/core';
-
-import { Observable } from 'rxjs/Observable';
-import { of }         from 'rxjs/observable/of';
-import { delay }      from 'rxjs/operators';
+import { Injectable } from '@angular/core';
 
 export class Crisis {
   constructor(public id: number, public name: string) { }
@@ -17,18 +13,18 @@ const CRISES: Crisis[] = [
 
 const FETCH_LATENCY = 500;
 
-/** Simulate a data service that retrieves crises from a server */
 @Injectable()
-export class CrisisService implements OnDestroy {
-  constructor() { console.log('CrisisService instance created.'); }
-  ngOnDestroy() { console.log('CrisisService instance destroyed.'); }
+export class CrisisService {
 
-  getCrises(): Observable<Crisis[]> {
-    return of(CRISES).pipe(delay(FETCH_LATENCY));
+  getCrises() {
+    return new Promise<Crisis[]>(resolve => {
+      setTimeout(() => { resolve(CRISES); }, FETCH_LATENCY);
+    });
   }
 
-  getCrisis(id: number | string): Observable<Crisis> {
-    return of(CRISES.find(crisis => crisis.id === +id))
-      .pipe(delay(FETCH_LATENCY));
+  getCrisis(id: number | string) {
+    return this.getCrises()
+      .then(heroes => heroes.find(hero => hero.id === +id));
   }
+
 }

aio/content/examples/ngmodule/src/app/hero/hero-detail.component.ts

@@ -26,6 +26,6 @@ export class HeroDetailComponent implements OnInit {
 
   ngOnInit() {
     let id = parseInt(this.route.snapshot.paramMap.get('id'), 10);
-    this.heroService.getHero(id).subscribe(hero => this.hero = hero);
+    this.heroService.getHero(id).then(hero => this.hero = hero);
   }
 }

aio/content/examples/ngmodule/src/app/hero/hero-list.component.ts

@@ -1,5 +1,4 @@
-import { Component }  from '@angular/core';
-import { Observable } from 'rxjs/Observable';
+import { Component, OnInit } from '@angular/core';
 
 import { Hero,
          HeroService } from './hero.service';
@@ -12,9 +11,11 @@ import { Hero,
     </div>
   `
 })
-export class HeroListComponent {
-  heroes: Observable<Hero[]>;
-  constructor(private heroService: HeroService) {
+export class HeroListComponent implements OnInit {
+  heroes: Promise<Hero[]>;
+  constructor(private heroService: HeroService) { }
+
+  ngOnInit() {
     this.heroes = this.heroService.getHeroes();
   }
 }

aio/content/examples/ngmodule/src/app/hero/hero.module.3.ts

@@ -5,9 +5,8 @@ import { FormsModule }         from '@angular/forms';
 import { HeroComponent }       from './hero.component.3';
 import { HeroDetailComponent } from './hero-detail.component';
 import { HeroListComponent }   from './hero-list.component';
-import { HeroRoutingModule }   from './hero-routing.module.3';
-
 import { HighlightDirective }  from './highlight.directive';
+import { HeroRoutingModule }   from './hero-routing.module.3';
 
 // #docregion class
 @NgModule({

aio/content/examples/ngmodule/src/app/hero/hero.service.ts

@@ -1,8 +1,4 @@
-import { Injectable, OnDestroy } from '@angular/core';
-
-import { Observable } from 'rxjs/Observable';
-import { of }         from 'rxjs/observable/of';
-import { delay }      from 'rxjs/operators';
+import { Injectable } from '@angular/core';
 
 export class Hero {
   constructor(public id: number, public name: string) { }
@@ -19,19 +15,18 @@ const HEROES: Hero[] = [
 
 const FETCH_LATENCY = 500;
 
-/** Simulate a data service that retrieves heroes from a server */
 @Injectable()
-export class HeroService implements OnDestroy {
+export class HeroService {
 
-  constructor() { console.log('HeroService instance created.'); }
-  ngOnDestroy() { console.log('HeroService instance destroyed.'); }
-
-  getHeroes(): Observable<Hero[]> {
-    return of(HEROES).pipe(delay(FETCH_LATENCY));
+  getHeroes() {
+    return new Promise<Hero[]>(resolve => {
+      setTimeout(() => { resolve(HEROES); }, FETCH_LATENCY);
+    });
   }
 
-  getHero(id: number | string): Observable<Hero> {
-    return of(HEROES.find(hero => hero.id === +id))
-      .pipe(delay(FETCH_LATENCY));
+  getHero(id: number | string) {
+    return this.getHeroes()
+      .then(heroes => heroes.find(hero => hero.id === +id));
   }
+
 }

aio/content/examples/ngmodule/src/app/highlight.directive.ts

@@ -1,15 +1,12 @@
-// #docplaster
 // #docregion
 import { Directive, ElementRef } from '@angular/core';
 
-// Highlight the host element in gold
 @Directive({ selector: '[highlight]' })
+/** Highlight the attached element in gold */
 export class HighlightDirective {
   constructor(el: ElementRef) {
     el.nativeElement.style.backgroundColor = 'gold';
-    // #enddocregion
-    console.log(`* AppRoot highlight called for ${el.nativeElement.tagName}`);
-    // #docregion
+    console.log(
+      `* AppRoot highlight called for ${el.nativeElement.tagName}`);
   }
 }
-// #enddocregion

aio/content/examples/ngmodule/src/app/shared/highlight.directive.ts

@@ -1,8 +1,9 @@
+/* tslint:disable */
 // Exact copy of contact/highlight.directive except for color and message
 import { Directive, ElementRef } from '@angular/core';
 
 @Directive({ selector: '[highlight], input' })
-// Highlight the host element or any InputElement in gray
+/** Highlight the attached element or an InputElement in gray */
 export class HighlightDirective {
   constructor(el: ElementRef) {
     el.nativeElement.style.backgroundColor = 'lightgray';

aio/content/examples/ngmodule/src/app/title.component.html

@@ -1,6 +1,6 @@
 <!-- #docregion -->
 <!-- #docregion v1 -->
-<h1 highlight>{{title}}</h1>
+<h1 highlight>{{title}} {{subtitle}}</h1>
 <!-- #enddocregion v1 -->
 <!-- #docregion ngIf -->
 <p *ngIf="user">

aio/content/examples/ngmodule/src/app/title.component.ts

@@ -1,17 +1,18 @@
 // #docplaster
 // #docregion
 // #docregion v1
-import { Component } from '@angular/core';
+import { Component, Input } from '@angular/core';
 // #enddocregion v1
 import { UserService } from './user.service';
 // #docregion v1
 
 @Component({
   selector: 'app-title',
-  templateUrl: './title.component.html'
+  templateUrl: './title.component.html',
 })
 export class TitleComponent {
-  title = 'Angular Modules';
+  @Input() subtitle = '';
+  title = 'NgModules';
 // #enddocregion v1
   user = '';
 

aio/content/examples/structural-directives/src/app/app.component.html

@@ -6,7 +6,7 @@
 
 <blockquote>
 <!-- #docregion built-in, asterisk, ngif -->
-<div *ngIf="hero" class="name">{{hero.name}}</div>
+<div *ngIf="hero" >{{hero.name}}</div>
 <!-- #enddocregion built-in, asterisk, ngif -->
 </blockquote>
 
@@ -51,7 +51,7 @@
 <p>&lt;ng-template&gt; element</p>
 <!-- #docregion ngif-template -->
 <ng-template [ngIf]="hero">
-  <div class="name">{{hero.name}}</div>
+  <div>{{hero.name}}</div>
 </ng-template>
 <!-- #enddocregion ngif-template -->
 

aio/content/examples/toh-pt4/src/app/message.service.ts

@@ -9,6 +9,6 @@ export class MessageService {
   }
 
   clear() {
-    this.messages = [];
+    this.messages.length = 0;
   }
 }

aio/content/examples/toh-pt5/src/app/message.service.ts

@@ -9,6 +9,6 @@ export class MessageService {
   }
 
   clear() {
-    this.messages = [];
+    this.messages.length = 0;
   }
 }

aio/content/examples/toh-pt6/src/app/message.service.ts

@@ -9,6 +9,6 @@ export class MessageService {
   }
 
   clear() {
-    this.messages = [];
+    this.messages.length = 0;
   }
 }

aio/content/examples/universal/src/app/message.service.ts

@@ -9,6 +9,6 @@ export class MessageService {
   }
 
   clear() {
-    this.messages = [];
+    this.messages.length = 0;
   }
 }

aio/content/guide/aot-compiler.md

@@ -409,7 +409,7 @@ function; it can only contain a single `return` statement.
 
 The Angular [`RouterModule`](api/router/RouterModule) exports two macro static methods, `forRoot` and `forChild`, to help declare root and child routes.
 Review the [source code](https://github.com/angular/angular/blob/master/packages/router/src/router_module.ts#L139 "RouterModule.forRoot source code")
-for these methods to see how macros can simplify configuration of complex [NgModules](guide/ngmodule).
+for these methods to see how macros can simplify configuration of complex Angular modules.
 
 ### Metadata rewriting
 

aio/content/guide/dependency-injection.md

@@ -160,7 +160,7 @@ Providing the `UserService` with an Angular module is a good choice.
 <div class="l-sub-section">
 
 To be precise, Angular module providers are registered with the root injector
-_unless the module is_ [lazy loaded](guide/ngmodule#lazy-load-DI).
+_unless the module is_ [lazy loaded](guide/ngmodule#lazy-load).
 In this sample, all modules are _eagerly loaded_ when the application starts,
 so all module providers are registered with the app's root injector.
 

aio/content/guide/quickstart.md

@@ -74,8 +74,8 @@ Generate a new project and skeleton application by running the following command
 
 
 
-Patience, please.
-It takes time to set up a new project; most of it is spent installing npm packages.
+Patience please.
+It takes time to set up a new project, most of it spent installing npm packages.
 
 
 </div>

aio/content/marketing/index.html

@@ -32,9 +32,9 @@
     <!-- Announcement Bar -->
     <div class="homepage-container">
       <div class="announcement-bar">
-        <img src="generated/images/marketing/home/ng-atl.png">
-        <p>Join us in Atlanta for ngATL<br>Jan 30 - Feb 2, 2018</p>
-        <a class="button" href="http://ng-atl.org/">Learn More</a>
+        <img src="generated/images/marketing/home/angular-connect.png">
+        <p>Join us in London for AngularConnect<br>November 7-8, 2017</p>
+        <a class="button" href="https://angularconnect.com/">Learn More</a>
       </div>
     </div>
 

aio/content/marketing/resources.json

@@ -241,12 +241,6 @@
       "UI Components": {
         "order": 4,
         "resources": {
-          "IgniteUIforAngular": {
-            "desc": "Ignite UI for Angular is a dependency-free Angular toolkit for building modern web apps.",
-            "rev": true,
-            "title": "Ignite UI for Angular",
-            "url": "https://www.infragistics.com/products/ignite-ui-angular"
-          },
           "DevExtreme": {
             "desc": "50+ UI components including data grid, pivot grid, scheduler, charts, editors, maps and other multi-purpose controls for creating highly responsive web applications for touch devices and traditional desktops.",
             "rev": true,
@@ -540,12 +534,6 @@
             "rev": true,
             "title": "Ultimate Angular",
             "url": "https://ultimateangular.com/"
-          },
-          "angular-firebase": {
-            "desc": "Video lessons covering progressive web apps with Angular, Firebase, RxJS, and related APIs.",
-            "rev": true,
-            "title": "AngularFirebase.com",
-            "url": "https://angularfirebase.com/"
           }
         }
       },
@@ -633,12 +621,6 @@
             "rev": true,
             "title": "SFEIR School (French)",
             "url": "https://school.sfeir.com/project/sa200/"
-          },
-          "formationjs": {
-            "desc": "Angular onsite training in Paris (France). Monthly Angular workshops and custom onsite classes. We are focused on Angular, so we are always up to date.",
-            "rev": true,
-            "title": "Formation JavaScript (French)",
-            "url": "https://formationjavascript.com/formation-angular/"
           }
         }
       }

aio/content/navigation.json

@@ -225,17 +225,17 @@
 
         {
           "title": "NgModules",
-          "tooltip": "Modularize your app with NgModules.",
+          "tooltip": "Learn how to use NgModules to make your apps efficient.",
           "children": [
             {
               "url": "guide/ngmodule",
               "title": "NgModules",
-              "tooltip": "Define application modules with the NgModule."
+              "tooltip": "Define application modules with @NgModule."
             },
             {
               "url": "guide/ngmodule-faq",
               "title": "NgModule FAQs",
-              "tooltip": "Answers to frequently asked questions about NgModules."
+              "tooltip": "Answers to frequently asked questions about @NgModule."
             }
         ]},
 

aio/content/tutorial/toh-pt4.md

@@ -3,7 +3,7 @@
 The Tour of Heroes `HeroesComponent` is currently getting and displaying fake data.
 
 After the refactoring in this tutorial, `HeroesComponent` will be lean and focused on supporting the view.
-It will also be easier to unit-test with a mock service.
+It will also be easier to unit-test with a mock services.
 
 ## Why services
 

aio/content/tutorial/toh-pt5.md

@@ -77,7 +77,7 @@ Then define an array of routes with a single `route` to that component.
 </code-example>
 
 Once you've finished setting up, the router will match that URL to `path: 'heroes'` 
-and display the `HeroesComponent`.
+and display the `HeroesComponent`, .
 
 ### _RouterModule.forRoot()_
 

aio/e2e/api.e2e-spec.ts

@@ -28,7 +28,7 @@ describe('Api pages', function() {
 
   it('should show readonly properties as getters', () => {
     const page = new ApiPage('api/common/http/HttpRequest');
-    expect(page.getOverview('class').getText()).toContain('get body: T | null');
+    expect(page.getOverview('class').getText()).toContain('get body: T|null');
   });
 
   it('should not show parenthesis for getters', () => {

aio/e2e/app.e2e-spec.ts

@@ -80,6 +80,8 @@ describe('site App', function() {
     });
   });
 
+  // TODO(https://github.com/angular/angular/issues/19785): Activate this again
+  // once it is no more flaky.
   describe('google analytics', () => {
 
     it('should call ga with initial URL', done => {

aio/e2e/app.po.ts

@@ -29,7 +29,10 @@ export class SitePage {
   locationPath() { return browser.executeScript('return document.location.pathname') as promise.Promise<string>; }
 
   navigateTo(pageUrl = '') {
-    return browser.get('/' + pageUrl);
+    return browser.get('/' + pageUrl)
+      // We need to tell the index.html not to load the real analytics library
+      // See the GA snippet in index.html
+      .then(() => browser.executeScript('sessionStorage.setItem("__e2e__", true);'));
   }
 
   getDocViewerText() {

aio/package.json

@@ -98,9 +98,8 @@
     "codelyzer": "~2.0.0",
     "concurrently": "^3.4.0",
     "cross-spawn": "^5.1.0",
-    "css-selector-parser": "^1.3.0",
     "dgeni": "^0.4.7",
-    "dgeni-packages": "0.22.1",
+    "dgeni-packages": "0.22.0",
     "entities": "^1.1.1",
     "eslint": "^3.19.0",
     "eslint-plugin-jasmine": "^2.2.0",

aio/scripts/_payload-limits.json

@@ -1,3 +0,0 @@
-{
-"aio":{"master":{"change":"application","gzip7":{"inline":925,"main":119519,"polyfills":11863},"gzip9":{"inline":925,"main":119301,"polyfills":11861},"uncompressed":{"inline":1533,"main":486493,"polyfills":37068}}}
-}

aio/scripts/_payload-limits.sh

@@ -0,0 +1,14 @@
+#!/bin/bash
+
+set -u -e -o pipefail
+
+declare -A payloadLimits
+payloadLimits["aio", "uncompressed", "inline"]=1600
+payloadLimits["aio", "uncompressed", "main"]=487000
+payloadLimits["aio", "uncompressed", "polyfills"]=38000
+payloadLimits["aio", "gzip7", "inline"]=1000
+payloadLimits["aio", "gzip7", "main"]=120000
+payloadLimits["aio", "gzip7", "polyfills"]=11900
+payloadLimits["aio", "gzip9", "inline"]=1000
+payloadLimits["aio", "gzip9", "main"]=120000
+payloadLimits["aio", "gzip9", "polyfills"]=11900

aio/scripts/payload.sh

@@ -7,6 +7,7 @@ readonly parentDir=$(dirname $thisDir)
 
 # Track payload size functions
 source ../scripts/ci/payload-size.sh
+source ${thisDir}/_payload-limits.sh
 
-trackPayloadSize "aio" "dist/*.bundle.js" true true "${thisDir}/_payload-limits.json"
+trackPayloadSize "aio" "dist/*.bundle.js" true true
 

aio/src/index.html

@@ -34,13 +34,11 @@
 
   <!-- Google Analytics -->
   <script>
-  // Note this is a customised version of the GA tracking snippet
-  // See the comments below for more info
+  // Note this is a customised version of the GA tracking snippet to aid e2e testing
+  // See the bit between /**/.../**/
   (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
   (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
-  m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;
-  ~i.name.indexOf('NG_DEFER_BOOTSTRAP')|| // only load library if not running e2e tests
-  m.parentNode.insertBefore(a,m)
+  m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;/**/i.sessionStorage.__e2e__||/**/m.parentNode.insertBefore(a,m)
   })(window,document,'script','https://www.google-analytics.com/analytics.js','ga');
   </script>
   <!-- End Google Analytics -->

aio/tools/example-zipper/exampleZipper.js

@@ -62,7 +62,7 @@ class ExampleZipper {
     let exampleZipName;
     const exampleType = this._getExampleType(path.join(sourceDirName, relativeDirName));
     if (relativeDirName.indexOf('/') !== -1) { // Special example
-      exampleZipName = relativeDirName.split('/').join('-');
+      exampleZipName = relativeDirName.split('/')[0];
     } else {
       exampleZipName = jsonFileName.replace(/(plnkr|zipper).json/, relativeDirName);
     }

aio/tools/examples/README.md

@@ -84,7 +84,7 @@ This tool expects all the examples to be build with `npm run build`. If an examp
 with that script, the author would need to specify the new build command in the `example-config.json`
 as shown earlier.
 
-### example-boilerplate.js
+### add-example-boilerplate.js
 
 This script installs all the dependencies that are shared among all the examples, creates the
 `node_modules` symlinks and copy all the boilerplate files where needed. It won't do anything
@@ -102,7 +102,7 @@ This script will find all the `e2e-spec.ts` files and run them.
 To not run all tests, you can use the `--filter=name` flag to run the example's e2e that contains
 that name.
 
-It also has an optional `--setup` flag to run the `example-boilerplate.js` script and install
+It also has an optional `--setup` flag to run the `add-example-boilerplate.js` script and install
 the latest `webdriver`.
 
 It will create a `/aio/protractor-results-txt` file when it finishes running tests.

aio/tools/examples/shared/boilerplate/cli/package.json

@@ -26,13 +26,13 @@
     "zone.js": "^0.8.14"
   },
   "devDependencies": {
-    "@angular/cli": "1.5.4",
+    "@angular/cli": "1.5.0",
     "@angular/compiler-cli": "^5.0.0",
     "@angular/language-service": "^5.0.0",
     "@types/jasmine": "~2.5.53",
     "@types/jasminewd2": "~2.0.2",
     "@types/node": "~6.0.60",
-    "codelyzer": "^4.0.1",
+    "codelyzer": "~3.2.0",
     "jasmine-core": "~2.6.2",
     "jasmine-spec-reporter": "~4.1.0",
     "karma": "~1.7.0",

aio/tools/transforms/angular-api-package/processors/addMetadataAliases.js

@@ -1,5 +1,3 @@
-const CssSelectorParser = require('css-selector-parser').CssSelectorParser;
-const cssParser = new CssSelectorParser();
 /**
  * @dgProcessor addMetadataAliases
  *
@@ -30,18 +28,11 @@ module.exports = function addMetadataAliasesProcessor() {
 };
 
 function extractSelectors(selectors) {
-  const selectorAST = cssParser.parse(stripQuotes(selectors));
-  const rules = selectorAST.selectors ? selectorAST.selectors.map(ruleSet => ruleSet.rule) : [selectorAST.rule];
-  const aliases = {};
-  rules.forEach(rule => {
-    if (rule.tagName) {
-      aliases[rule.tagName] = true;
-    }
-    if (rule.attrs) {
-      rule.attrs.forEach(attr => aliases[attr.name] = true);
-    }
-  });
-  return Object.keys(aliases);
+  if (selectors) {
+    return stripQuotes(selectors).split(',').map(selector => selector.replace(/^\W*([\w-]+)\W*$/, '$1'));
+  } else {
+    return [];
+  }
 }
 
 function stripQuotes(value) {

aio/tools/transforms/angular-api-package/processors/addMetadataAliases.spec.js

@@ -45,8 +45,8 @@ describe('addSelectorsAsAliases processor', () => {
     expect(docs[2].aliases).toEqual([docs[2].name]);
     expect(docs[3].aliases).toEqual([docs[3].name]);
     expect(docs[4].aliases).toEqual([docs[4].name, 'myPipe']);
-    expect(docs[5].aliases).toEqual([docs[5].name, 'my-directive', 'myDirective']);
-    expect(docs[6].aliases).toEqual([docs[6].name, 'ngModel']);
+    expect(docs[5].aliases).toEqual([docs[5].name, 'my-directive', 'myDirective', 'my-directive']);
+    expect(docs[6].aliases).toEqual([docs[6].name, '[ngModel]:not([formControlName]):not([formControl])']);
     expect(docs[7].aliases).toEqual([docs[7].name, 'my-component']);
     expect(docs[8].aliases).toEqual([docs[8].name]);
     expect(docs[9].aliases).toEqual([docs[9].name]);

aio/tools/transforms/angular-base-package/index.js

@@ -35,8 +35,6 @@ module.exports = new Package('angular-base', [
   .factory('packageInfo', function() { return require(path.resolve(PROJECT_ROOT, 'package.json')); })
   .factory(require('./readers/json'))
   .factory(require('./services/copyFolder'))
-  .factory(require('./services/filterPipes'))
-  .factory(require('./services/filterAmbiguousDirectiveAliases'))
   .factory(require('./services/getImageDimensions'))
 
   .factory(require('./post-processors/add-image-dimensions'))
@@ -128,9 +126,8 @@ module.exports = new Package('angular-base', [
   })
 
 
-  .config(function(postProcessHtml, addImageDimensions, autoLinkCode, filterPipes, filterAmbiguousDirectiveAliases) {
+  .config(function(postProcessHtml, addImageDimensions, autoLinkCode) {
     addImageDimensions.basePath = path.resolve(AIO_PATH, 'src');
-    autoLinkCode.customFilters = [filterPipes, filterAmbiguousDirectiveAliases];
     postProcessHtml.plugins = [
       require('./post-processors/autolink-headings'),
       addImageDimensions,

aio/tools/transforms/angular-base-package/post-processors/auto-link-code.js

@@ -10,19 +10,12 @@ const textContent = require('hast-util-to-string');
  * Only docs that have one of these docTypes will be linked to.
  * Usually set to the API exported docTypes, e.g. "class", "function", "directive", etc.
  *
- * @property customFilters array of functions `(docs, words, wordIndex) => docs` that will filter
- * out docs where a word should not link to a doc.
- *   - `docs` is the array of docs that match the link `word`
- *   - `words` is the collection of words parsed from the code text
- *   - `wordIndex` is the index of the current `word` for which we are finding a link
- *
  * @property codeElements an array of strings.
  * Only text contained in these elements will be linked to.
  * Usually set to "code" but also "code-example" for angular.io.
  */
 module.exports = function autoLinkCode(getDocFromAlias) {
   autoLinkCodeImpl.docTypes = [];
-  autoLinkCodeImpl.customFilters = [];
   autoLinkCodeImpl.codeElements = ['code'];
   return autoLinkCodeImpl;
 
@@ -45,13 +38,12 @@ module.exports = function autoLinkCode(getDocFromAlias) {
                 parent.children.splice(index, 1, createLinkNode(docs[0], node.value));
               } else {
                 // Parse the text for words that we can convert to links
-                const nodes = textContent(node).split(/([A-Za-z0-9_-]+)/)
+                const nodes = textContent(node).split(/([A-Za-z0-9_]+)/)
                   .filter(word => word.length)
-                  .map((word, index, words) => {
-                    // remove docs that fail the custom filter tests
-                    const filteredDocs = autoLinkCodeImpl.customFilters.reduce((docs, filter) => filter(docs, words, index), getDocFromAlias(word));
-                    return foundValidDoc(filteredDocs) ?
-                              createLinkNode(filteredDocs[0], word) : // Create a link wrapping the text node.
+                  .map(word => {
+                    const docs = getDocFromAlias(word);
+                    return foundValidDoc(docs) ?
+                              createLinkNode(docs[0], word) : // Create a link wrapping the text node.
                               { type: 'text', value: word };  // this is just text so push a new text node
                   });
 

aio/tools/transforms/angular-base-package/post-processors/auto-link-code.spec.js

@@ -2,7 +2,7 @@ var createTestPackage = require('../../helpers/test-package');
 var Dgeni = require('dgeni');
 
 describe('autoLinkCode post-processor', () => {
-  let processor, autoLinkCode, aliasMap, filterPipes;
+  let processor, autoLinkCode, aliasMap;
 
   beforeEach(() => {
     const testPackage = createTestPackage('angular-base-package');
@@ -14,7 +14,6 @@ describe('autoLinkCode post-processor', () => {
     processor = injector.get('postProcessHtml');
     processor.docTypes = ['test-doc'];
     processor.plugins = [autoLinkCode];
-    filterPipes = injector.get('filterPipes');
   });
 
   it('should insert an anchor into every code item that matches the id of an API doc', () => {
@@ -52,26 +51,6 @@ describe('autoLinkCode post-processor', () => {
     expect(doc.renderedContent).toEqual('<code>MyClass</code>');
   });
 
-  it('should ignore code items that match an API doc but are attached to other text via a dash', () => {
-    aliasMap.addDoc({ docType: 'class', id: 'MyClass', aliases: ['MyClass'], path: 'a/b/myclass' });
-    const doc = { docType: 'test-doc', renderedContent: '<code>xyz-MyClass</code>' };
-    processor.$process([doc]);
-    expect(doc.renderedContent).toEqual('<code>xyz-MyClass</code>');
-  });
-
-  it('should ignore code items that are filtered out by custom filters', () => {
-    autoLinkCode.customFilters = [filterPipes];
-    aliasMap.addDoc({ docType: 'pipe', id: 'MyClass', aliases: ['MyClass', 'myClass'], path: 'a/b/myclass', pipeOptions: { name: '\'myClass\'' } });
-    const doc = { docType: 'test-doc', renderedContent: '<code>{ xyz | myClass } { xyz|myClass } MyClass myClass OtherClass|MyClass</code>' };
-    processor.$process([doc]);
-    expect(doc.renderedContent).toEqual('<code>' +
-                                        '{ xyz | <a href="a/b/myclass" class="code-anchor">myClass</a> } '  +
-                                        '{ xyz|<a href="a/b/myclass" class="code-anchor">myClass</a> } ' +
-                                        '<a href="a/b/myclass" class="code-anchor">MyClass</a> ' +
-                                        'myClass OtherClass|<a href="a/b/myclass" class="code-anchor">MyClass</a>' +
-                                        '</code>');
-  });
-
   it('should insert anchors for individual text nodes within a code block', () => {
     aliasMap.addDoc({ docType: 'class', id: 'MyClass', aliases: ['MyClass'], path: 'a/b/myclass' });
     const doc = { docType: 'test-doc', renderedContent: '<code><span>MyClass</span><span>MyClass</span></code>' };

aio/tools/transforms/angular-base-package/services/filterAmbiguousDirectiveAliases.js

@@ -1,26 +0,0 @@
-
-/**
- * This service is used by the autoLinkCode post-processor to filter out ambiguous directive
- * docs where the matching word is a directive selector.
- * E.g. `ngModel`, which is a selector for a number of directives, where we are only really
- * interested in the `NgModel` class.
- */
-module.exports = function filterAmbiguousDirectiveAliases() {
-  return (docs, words, index) => {
-    const word = words[index];
-
-    // we are only interested if there are multiple matching docs
-    if (docs.length > 1) {
-      if (docs.every(doc =>
-        // We are only interested if they are all either directives or components
-        (doc.docType === 'directive' || doc.docType === 'component') &&
-        // and the matching word is in the selector for all of them
-        doc[doc.docType + 'Options'].selector.indexOf(word) != -1
-      )) {
-        // find the directive whose class name matches the word (case-insensitive)
-        return docs.filter(doc => doc.name.toLowerCase() === word.toLowerCase());
-      }
-    }
-    return docs;
-  };
-};

aio/tools/transforms/angular-base-package/services/filterAmbiguousDirectiveAliases.spec.js

@@ -1,50 +0,0 @@
-const filterAmbiguousDirectiveAliases = require('./filterAmbiguousDirectiveAliases')();
-
-const words = ['Http', 'ngModel', 'NgModel', 'NgControlStatus'];
-
-describe('filterAmbiguousDirectiveAliases(docs, words, index)', () => {
-  it('should not try to filter the docs, if the docs are not all directives or components', () => {
-    const docs = [
-      { docType: 'class', name: 'Http' },
-      { docType: 'directive', name: 'NgModel', directiveOptions: { selector: '[ngModel]' } },
-      { docType: 'component', name: 'NgModel', componentOptions: { selector: '[ngModel]' } }
-    ];
-    // take a copy to prove `docs` was not modified
-    const filteredDocs = docs.slice(0);
-    expect(filterAmbiguousDirectiveAliases(docs, words, 1)).toEqual(filteredDocs);
-    expect(filterAmbiguousDirectiveAliases(docs, words, 2)).toEqual(filteredDocs);
-  });
-
-  describe('(where all the docs are components or directives', () => {
-    describe('and do not all contain the matching word in their selector)', () => {
-      it('should not try to filter the docs', () => {
-        const docs = [
-          { docType: 'directive', name: 'NgModel', ['directiveOptions']: { selector: '[ngModel]' } },
-          { docType: 'component', name: 'NgControlStatus', ['componentOptions']: { selector: '[ngControlStatus]' } }
-        ];
-        // take a copy to prove `docs` was not modified
-        const filteredDocs = docs.slice(0);
-        expect(filterAmbiguousDirectiveAliases(docs, words, 1)).toEqual(filteredDocs);
-        expect(filterAmbiguousDirectiveAliases(docs, words, 2)).toEqual(filteredDocs);
-
-        // Also test that the check is case-sensitive
-        docs[1].componentOptions.selector = '[ngModel]';
-        filteredDocs[1].componentOptions.selector = '[ngModel]';
-        expect(filterAmbiguousDirectiveAliases(docs, words, 2)).toEqual(filteredDocs);
-      });
-    });
-
-    describe('and do all contain the matching word in there selector)', () => {
-      it('should filter out docs whose class name is not (case-insensitively) equal to the matching word', () => {
-        const docs = [
-          { docType: 'directive', name: 'NgModel', ['directiveOptions']: { selector: '[ngModel],[ngControlStatus]' } },
-          { docType: 'component', name: 'NgControlStatus', ['componentOptions']: { selector: '[ngModel],[ngControlStatus]' } }
-        ];
-        const filteredDocs = [
-          { docType: 'directive', name: 'NgModel', ['directiveOptions']: { selector: '[ngModel],[ngControlStatus]' } }
-        ];
-        expect(filterAmbiguousDirectiveAliases(docs, words, 1)).toEqual(filteredDocs);
-      });
-    });
-  });
-});

aio/tools/transforms/angular-base-package/services/filterPipes.js

@@ -1,12 +0,0 @@
-
-/**
- * This service is used by the autoLinkCode post-processors to filter out pipe docs
- * where the matching word is the pipe name and is not preceded by a pipe
- */
-module.exports = function filterPipes() {
-  return (docs, words, index) =>
-    docs.filter(doc =>
-      doc.docType !== 'pipe' ||
-      doc.pipeOptions.name !== '\'' + words[index] + '\'' ||
-      index > 0 && words[index - 1].trim() === '|');
-};

aio/tools/transforms/angular-base-package/services/filterPipes.spec.js

@@ -1,36 +0,0 @@
-const filterPipes = require('./filterPipes')();
-
-describe('filterPipes', () => {
-  it('should ignore docs that are not pipes', () => {
-    const docs = [{ docType: 'class', name: 'B', pipeOptions: { name: '\'b\'' } }];
-    const words = ['A', 'b', 'B', 'C'];
-    const filteredDocs = [{ docType: 'class', name: 'B', pipeOptions: { name: '\'b\'' } }];
-    expect(filterPipes(docs, words, 1)).toEqual(filteredDocs);
-    expect(filterPipes(docs, words, 2)).toEqual(filteredDocs);
-  });
-
-  it('should ignore docs that are pipes but do not match the pipe name', () => {
-    const docs = [{ docType: 'pipe', name: 'B', pipeOptions: { name: '\'b\'' } }];
-    const words = ['A', 'B', 'C'];
-    const filteredDocs = [{ docType: 'pipe', name: 'B', pipeOptions: { name: '\'b\'' } }];
-    expect(filterPipes(docs, words, 1)).toEqual(filteredDocs);
-  });
-
-  it('should ignore docs that are pipes, match the pipe name and are preceded by a pipe character', () => {
-    const docs = [{ docType: 'pipe', name: 'B', pipeOptions: { name: '\'b\'' } }];
-    const words = ['A', '|', 'b', 'C'];
-    const filteredDocs = [{ docType: 'pipe', name: 'B', pipeOptions: { name: '\'b\'' } }];
-    expect(filterPipes(docs, words, 2)).toEqual(filteredDocs);
-  });
-
-  it('should filter out docs that are pipes, match the pipe name but are not preceded by a pipe character', () => {
-    const docs = [
-      { docType: 'pipe', name: 'B', pipeOptions: { name: '\'b\'' } },
-      { docType: 'class', name: 'B' }
-    ];
-    const words = ['A', 'b', 'C'];
-    const index = 1;
-    const filteredDocs = [{ docType: 'class', name: 'B' }];
-    expect(filterPipes(docs, words, index)).toEqual(filteredDocs);
-  });
-});

aio/yarn.lock

@@ -2007,10 +2007,6 @@ css-select@^1.1.0:
     domutils "1.5.1"
     nth-check "~1.0.1"
 
-css-selector-parser@^1.3.0:
-  version "1.3.0"
-  resolved "https://registry.yarnpkg.com/css-selector-parser/-/css-selector-parser-1.3.0.tgz#5f1ad43e2d8eefbfdc304fcd39a521664943e3eb"
-
 css-selector-tokenizer@^0.7.0:
   version "0.7.0"
   resolved "https://registry.yarnpkg.com/css-selector-tokenizer/-/css-selector-tokenizer-0.7.0.tgz#e6988474ae8c953477bf5e7efecfceccd9cf4c86"
@@ -2287,9 +2283,9 @@ devtools-timeline-model@1.1.6:
     chrome-devtools-frontend "1.0.401423"
     resolve "1.1.7"
 
-dgeni-packages@0.22.1:
-  version "0.22.1"
-  resolved "https://registry.yarnpkg.com/dgeni-packages/-/dgeni-packages-0.22.1.tgz#c4587a765689c4c9d48ed661517ed2249403bfb2"
+dgeni-packages@0.22.0:
+  version "0.22.0"
+  resolved "https://registry.yarnpkg.com/dgeni-packages/-/dgeni-packages-0.22.0.tgz#7ed07af9074f6547847256c1a65b488a5a17ad03"
   dependencies:
     canonical-path "0.0.2"
     catharsis "^0.8.1"

docs/TRIAGE_AND_LABELS.md

@@ -1,52 +1,49 @@
 # Triage Process and GitHub Labels for Angular
 
-This document describes how the Angular team uses labels and milestones to triage issues on github.
-The basic idea of the process is that caretaker only assigns a component (`comp: *`) label.
-The owner of the component is then responsible for the secondary / component-level triage.
+This document describes how the Angular team uses labels and milestones
+to triage issues on github. The basic idea of the process is that
+caretaker only assigns a component and type (bug, feature) label. The
+owner of the component than is in full control of how the issues should
+be triaged further.
 
+Once this process is implemented and in use, we will revisit it to see
+if further labeling is needed.
 
 ## Label Types
 
 ### Components
 
-The caretaker should be able to determine which component the issue belongs to.
-The components have a clear piece of source code associated with it within the `/packages/` folder of this repo.
+A caretaker should be able to determine which component the issue
+belongs to. The components have a clear piece of source code associated
+with it within the `/packages/` folder of this repo.
 
 * `comp: aio` - the angular.io application
 * `comp: animations`
-* `comp: bazel` - @angular/bazel rules
+* `comp: bazel`
 * `comp: benchpress`
 * `comp: common` - this includes core components / pipes
-* `comp: common/http` - this includes core components / pipes
-* `comp: core & compiler` - because core, compiler, compiler-cli and
+* `comp: core, compiler` - because core, compiler, compiler-cli and
   browser-platforms are very intertwined, we will be treating them as one
 * `comp: forms`
 * `comp: http`
 * `comp: i18n`
-* `comp: language-service`
-* `comp: metadata-extractor`
+* `comp: language service`
 * `comp: router`
-* `comp: server`
-* `comp: service-worker`
 * `comp: testing`
-* `comp: upgrade/dynamic`
-* `comp: upgrade/static`
+* `comp: upgrade`
 * `comp: web-worker`
 * `comp: zones`
 
-There are few components which are cross-cutting.
-They don't have a clear location in the source tree.
-We will treat them as a component even thought no specific source tree is associated with them.
+There are few components which are cross-cutting. They don't have
+a clear location in the source tree. We will treat them as a component
+even thought no specific source tree is associated with them.
 
-* `comp: build & ci` - build and CI infrastructure for the angular/angular repo
+* `comp: build & ci` - all build and CI scripts
 * `comp: docs` - documentation, including API docs, guides, tutorial
-* `comp: packaging` - packaging format of @angular/* npm packages
+* `comp: packaging`
 * `comp: performance`
 * `comp: security`
 
-Sometimes, especially in the case of cross-cutting issues or PRs, these PRs or issues belong to multiple components.
-In these cases, all applicable component labels should be used to triage the issue or PR.
-
 
 ### Type
 
@@ -59,27 +56,27 @@ What kind of problem is this?
 * `type: performance`
 * `type: refactor`
 
+## Caretaker Triage Process
 
-## Caretaker Triage Process (Primary Triage)
-
-It is the caretaker's responsibility to assign `comp:  *` to each new issue as they come in.
-
-If it's obvious that the issue or PR is related to a release regression, the caretaker is also responsible for assigning the `severity(5): regression` label to make the issue or PR highly visible.
-
-The primary triage should be done on a daily basis so that the issues become available for secondary triage without much of delay.
-
-The reason why we limit the responsibility of the caretaker to this one label is that it is likely that without domain knowledge the caretaker could mislabel issues or lack knowledge of duplicate issues.
+It is the caretaker's responsibility to assign `comp:  *` to each new
+issue as they come in. The reason why we limit the responsibility of the
+caretaker to this one label is that it is likely that without domain
+knowledge the caretaker could mislabel issues or lack knowledge of
+duplicate issues.
 
 
 ## Component's owner Triage Process
 
-The component owner is responsible for assigning one of the labels from each of these categories:
+At this point we are leaving each component owner to determine their own
+process for their component.
 
-- `type: *`
-- `frequency: *`
-- `severity: *`
+It will be up to the component owner to determine the order in which the
+issues within the component will be resolved.
 
-We've adopted the issue categorization based on [user pain](http://www.lostgarden.com/2008/05/improving-bug-triage-with-user-pain.html) used by AngularJS. In this system every issue is assigned frequency and severity based on which the total user pain score is calculated.
+Several owners have adopted the issue categorization based on
+[user pain](http://www.lostgarden.com/2008/05/improving-bug-triage-with-user-pain.html)
+used by AngularJS. In this system every issue is assigned frequency and
+severity based on which the total user pain score is calculated.
 
 Following is the definition of various frequency and severity levels:
 
@@ -101,42 +98,42 @@ These criteria are then used to calculate a "user pain" score as follows:
 
 `pain = severity × frequency`
 
-This score can then be used to estimate the impact of the issue which helps with prioritization.
 
+## Triaged vs Untrained PRs
 
-## Triaging PRs
+PRs should also be label with a `comp: *` so that it is clear which
+primary area the PR effects.
 
-Triaging PRs is the same as triaging issues, except that PRs have additional label categories that should be used to signal their state.
+Because of the cumulative pain associated with rebasing PRs, we triage PRs daily, and
+closing or reviewing PRs is a top priority ahead of other ongoing work.
 
-Every triaged PR must have a `pr_action` label assigned to it:
+Every triaged PR must have a `pr_action` label assigned to it and an assignee:
 
-* `PR action: review` - work is complete and comment is needed from the reviewers.
-* `PR action: cleanup` - more work is needed from the author.
-* `PR action: discuss` - discussion is needed, to be led by the author.
+* `PR action: review` - work is complete and comment is needed from the assignee.
+* `PR action: cleanup` - more work is needed from the current assignee.
+* `PR action: discuss` - discussion is needed, to be led by the current assignee.
 * `PR action: merge` - the PR is ready to be merged by the caretaker.
 
 In addition, PRs can have the following states:
 
 * `PR state: WIP` - PR is experimental or rapidly changing. Not ready for review or triage.
-* `PR state: blocked` - PR is blocked on an issue or other PR. Not ready for review or triage or merge.
+* `PR state: blocked` - PR is blocked on an issue or other PR. Not ready for review or triage.
 
 
 ## PR Target
 
-In our git workflow, we merge changes either to the `master` branch, the active patch branch (e.g. `5.0.x`), or to both.
+In our git workflow, we merge changes either to the `master` branch, the most recent patch branch (e.g. `4.3.x`), or to both.
 
-The decision about the target must be done by the PR author and/or reviewer.
-This decision is then honored when the PR is being merged by the caretaker.
+The decision about the target must be done by the PR author and/or reviewer. This decision is then honored when the PR is being merged.
 
 To communicate the target we use the following labels:
 
-* `PR target: master & patch`: the PR should me merged into the master branch and cherry-picked into the most recent patch branch. All PRs with fixes, docs and refactorings should use this target.
-* `PR target: master-only`: the PR should be merged only into the `master` branch. All PRs with new features, API changes or high-risk changes should use this target.
-* `PR target: patch-only`: the PR should be merged only into the most recent patch branch (e.g. 5.0.x). This target is useful if a `master & patch` PR can't be cleanly cherry-picked into the stable branch and a new PR is needed.
-* `PR target: LTS-only`: the PR should be merged only into the active LTS branch(es). Only security and critical fixes are allowed in these branches. Always send a new PR targeting just the LTS branch and request review approval from @IgorMinar.
-* `PR target: TBD`: the target is yet to be determined.
+* `PR target: master-only`
+* `PR target: patch-only`
+* `PR target: master & patch`
+* `PR target: TBD` - the target is yet to be determined
 
-If a PR is missing the "PR target: *" label, or if the label is set to "TBD" when the PR is sent to the caretaker, the caretaker should reject the PR and request the appropriate target label to be applied before the PR is merged.
+If a PR is missing the "PR target" label, or if the label is set to "TBD" when the PR is sent to the caretaker, the caretaker should reject the PR and request the appropriate target label to be applied before the PR is merged.
 
 
 ## PR Approvals
@@ -145,17 +142,23 @@ Before a PR can be merged it must be approved by the appropriate reviewer(s).
 
 To ensure that there right people review each change, we configured [PullApprove bot](https://about.pullapprove.com/) via (`.pullapprove.yaml`) to provide aggregate approval state via the GitHub PR Status API.
 
-Note that approved state does not mean a PR is ready to be merged.
-For example, a reviewer might approve the PR but request a minor tweak that doesn't need further review, e.g., a rebase or small uncontroversial change.
-Only the `PR action: merge` label means that the PR is ready for merging.
+Note that approved state does not mean a PR is ready to be merged. For example, a reviewer might
+approve the PR but request a minor tweak that doesn't need further review, e.g., a rebase or small
+uncontroversial change.
 
 
 ## Special Labels
 
-### `cla: yes`, `cla: no`
-Managed by googlebot.
-Indicates whether a PR has a CLA on file for its author(s).
-Only issues with `cla:yes` should be merged into master.
+### action:design
+More active discussion is needed before the issue can be worked on further. Typically used for
+`type: feature` or `type: RFC/discussion/question`
 
-### `aio: preview`
-Applying this label to a PR makes the angular.io preview available regardless of the author. [More info](../aio/aio-builds-setup/docs/overview--security-model.md)
+[See all issues that need discussion](https://github.com/angular/angular/labels/action:%20Design)
+
+### cla: yes, cla: no
+Managed by googlebot. Indicates whether a PR has a CLA on file for its author(s). Only issues with
+`cla:yes` should be merged into master.
+
+### WORKS_AS_INTENDED
+
+Only used on closed issues, to indicate to the reporter why we closed it.

integration/_payload-limits.json

@@ -1,4 +0,0 @@
-{
-"cli-hello-world":{"master":{"gzip7":{"inline":847,"main":42533,"polyfills":20207},"gzip9":{"inline":847,"main":42483,"polyfills":20204},"uncompressed":{"inline":1447,"main":154295,"polyfills":61254}}},
-"hello_world__closure":{"master":{"gzip7":{"bundle":32793},"gzip9":{"bundle":32758},"uncompressed":{"bundle":100661}}}
-}

integration/_payload-limits.sh

@@ -0,0 +1,19 @@
+#!/bin/bash
+
+set -u -e -o pipefail
+
+declare -A payloadLimits
+payloadLimits["hello_world__closure", "uncompressed", "bundle"]=106000
+payloadLimits["hello_world__closure", "gzip7", "bundle"]=35000
+payloadLimits["hello_world__closure", "gzip9", "bundle"]=35000
+
+payloadLimits["cli-hello-world", "uncompressed", "inline"]=1500
+payloadLimits["cli-hello-world", "uncompressed", "main"]=160000
+payloadLimits["cli-hello-world", "uncompressed", "polyfills"]=66000
+payloadLimits["cli-hello-world", "gzip7", "inline"]=900
+payloadLimits["cli-hello-world", "gzip7", "main"]=45000
+payloadLimits["cli-hello-world", "gzip7", "polyfills"]=22000
+payloadLimits["cli-hello-world", "gzip9", "inline"]=900
+payloadLimits["cli-hello-world", "gzip9", "main"]=45000
+payloadLimits["cli-hello-world", "gzip9", "polyfills"]=22000
+

integration/bazel/WORKSPACE

@@ -3,7 +3,7 @@ load("@bazel_tools//tools/build_defs/repo:git.bzl", "git_repository")
 git_repository(
     name = "build_bazel_rules_nodejs",
     remote = "https://github.com/bazelbuild/rules_nodejs.git",
-    tag = "0.2.1",
+    tag = "0.0.2",
 )
 
 load("@build_bazel_rules_nodejs//:defs.bzl", "node_repositories")

integration/dynamic-compiler/e2e/app.e2e-spec.ts

@@ -1,19 +0,0 @@
-import { AppPage } from './app.po';
-
-describe('dynamic-compiler App', () => {
-  let page: AppPage;
-
-  beforeEach(() => {
-    page = new AppPage();
-  });
-
-  it('should display welcome message', () => {
-    page.navigateTo();
-    expect(page.getParagraphText()).toEqual('Hello world!');
-  });
-
-  it('should display lazy-loaded component', () => {
-    page.navigateTo();
-    expect(page.getLazyLoadedText()).toEqual('Lazy-loaded component!');
-  });
-});

integration/dynamic-compiler/e2e/app.po.ts

@@ -1,16 +0,0 @@
-import {browser, by, element, protractor} from 'protractor';
-
-export class AppPage {
-  navigateTo() {
-    return browser.get('/');
-  }
-
-  getParagraphText() {
-    return element(by.css('app-root h1')).getText();
-  }
-
-  getLazyLoadedText() {
-    const el = element(by.css('app-root lazy-component'));
-    return browser.wait(protractor.ExpectedConditions.presenceOf(el)).then(() => el.getText(), err => el.getText());
-  }
-}