Tsconfig migration fixes #16050
Tsconfig migration fixes #16050
Conversation
…*.d.ts` This fixes warnings such as the below; ``` WARNING in /test-update/src/environments/environment.prod.ts is part of the TypeScript compilation but it's unused. Add only entry points to the 'files' or 'include' properties in your tsconfig. ``` When the previous generated add was in VE.
alan-agius4
added
the
target: patch
| expect(include).toEqual(['foo.ts']); | ||
| }); | ||
|
|
||
| it(`should update include '**/*.ts' in apps tsConfig to '**/*.d.ts'`, async () => { |
There was a problem hiding this comment.
This migration will cause projects that use the string format of loadChildren with broken compilations, because the lazy routes will no longer be part of the compilation.
That being said, we have already deprecated the string syntax and migrated these as part of the previous version 8 migration so it should be ok. If users have trouble with this we can provide instructions on how to add the lazy routes to the compilation for users that need to still use the string syntax.
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
See individual commits.