-
Notifications
You must be signed in to change notification settings - Fork 24.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(compiler-cli): support multiple configuration files in extends
#49125
Conversation
Blocked on #49107 |
f95b4c1
to
7785571
Compare
TypeScript 5 support `extends` to be an array, this commit adds support to allow extending `angularCompilerOptions` from multiple config files. See: https://devblogs.microsoft.com/typescript/announcing-typescript-5-0-beta/#supporting-multiple-configuration-files-in-extends
7785571
to
602f492
Compare
@@ -194,4 +194,52 @@ describe('perform_compile', () => { | |||
debug: false, | |||
})); | |||
}); | |||
|
|||
it('should merge tsconfig "angularCompilerOptions" when extends is an array', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder what the behavior here should be for nested options, extendedDiagnostics
being the primary example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That doesn’t change with this change. Complex objects such as paths
and extendedDiagnostics
ste not merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a test to that effect? I realise it's more about extends
in general than having multiple extends
per se, but wouldn't hurt to explicitly capture this behavior in a test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do not, but I can add it if you wish.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Capturing that behavior would be valuable, IMO
@@ -194,4 +194,52 @@ describe('perform_compile', () => { | |||
debug: false, | |||
})); | |||
}); | |||
|
|||
it('should merge tsconfig "angularCompilerOptions" when extends is an array', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Capturing that behavior would be valuable, IMO
This commit adds a test case for validate that we do not deep merge objects like like 'paths' and `extendedDiagnostics`.
This PR was merged into the repository by commit 1418d19. |
This commit adds a test case for validate that we do not deep merge objects like like 'paths' and `extendedDiagnostics`. PR Close #49125
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. |
TypeScript 5 support
extends
to be an array, this commit adds support to allow extendingangularCompilerOptions
from multiple config files.See: https://devblogs.microsoft.com/typescript/announcing-typescript-5-0-beta/#supporting-multiple-configuration-files-in-extends