angular-cn/packages/core/schematics/migrations
Paul Gschwendtner 28995dba19 fix(core): missing-injectable migration should not migrate `@NgModule` classes (#36369)
Based on the migration guide, provided classes which don't have
either `@Injectable`, `@Directive`, `@Component` or `@Pipe` need
to be migrated.

This is not correct as provided classes with an `@NgModule` also
have a factory function that can be read by the r3 injector. It's
unclear in which cases the `@NgModule` decorator is used for
provided classes, but this scenario has been reported.

Either we fix this in the migration, or we make sure to report
this as unsupported in the Ivy compiler.

Fixes #35700.

PR Close #36369
2020-04-21 12:54:24 -04:00
..
dynamic-queries fix(core): avoid migration error when non-existent symbol is imported (#36367) 2020-04-06 13:21:54 -07:00
google3 build: reformat repo to new clang@1.4.0 (#36613) 2020-04-14 12:08:36 -07:00
missing-injectable fix(core): missing-injectable migration should not migrate `@NgModule` classes (#36369) 2020-04-21 12:54:24 -04:00
module-with-providers build: reformat repo to new clang@1.4.0 (#36613) 2020-04-14 12:08:36 -07:00
move-document fix(core): avoid migration error when non-existent symbol is imported (#36367) 2020-04-06 13:21:54 -07:00
renderer-to-renderer2 build: reformat repo to new clang@1.4.0 (#36613) 2020-04-14 12:08:36 -07:00
static-queries build: reformat repo to new clang@1.4.0 (#36613) 2020-04-14 12:08:36 -07:00
template-var-assignment fix(core): avoid migration error when non-existent symbol is imported (#36367) 2020-04-06 13:21:54 -07:00
undecorated-classes-with-decorated-fields build: reformat repo to new clang@1.4.0 (#36613) 2020-04-14 12:08:36 -07:00
undecorated-classes-with-di build: reformat repo to new clang@1.4.0 (#36613) 2020-04-14 12:08:36 -07:00