Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Oct 1, 2025

The unit-test builder is updated to streamline the initial configuration by intelligently defaulting the tsConfig option.

If the tsConfig option is not provided in the project's configuration, the builder now checks for the existence of a tsconfig.spec.json file in the project's root. If found, it is used automatically. This removes the need for boilerplate tsConfig configuration in angular.json for the majority of projects.

Additionally, a validation check has been added to ensure that if a tsConfig path is specified, the file must exist. An actionable error is thrown if it is not found, preventing downstream build failures.

The builder's schema has been updated to reflect these changes, making the tsConfig option optional and documenting the new behavior.

@clydin clydin added the target: major This PR is targeted for the next major release label Oct 1, 2025
@clydin clydin force-pushed the unit-test/default-tsconfig branch from dc5ffb7 to 42786fc Compare October 1, 2025 20:38
The unit-test builder is updated to streamline the initial configuration by intelligently defaulting the `tsConfig` option.

If the `tsConfig` option is not provided in the project's configuration, the builder now checks for the existence of a `tsconfig.spec.json` file in the project's root. If found, it is used automatically. This removes the need for boilerplate `tsConfig` configuration in `angular.json` for the majority of projects.

Additionally, a validation check has been added to ensure that if a `tsConfig` path is specified, the file must exist. An actionable error is thrown if it is not found, preventing downstream build failures.

The builder's schema has been updated to reflect these changes, making the `tsConfig` option optional and documenting the new behavior.
@clydin clydin force-pushed the unit-test/default-tsconfig branch from 42786fc to 79a53cb Compare October 1, 2025 21:02
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 1, 2025
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 2, 2025
@clydin clydin merged commit 7761180 into angular:main Oct 2, 2025
57 of 58 checks passed
@clydin clydin deleted the unit-test/default-tsconfig branch October 2, 2025 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants