Skip to content

fix(@angular/build): ensure disabled vitest config loading #30351

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

Merged
merged 1 commit into from
May 21, 2025

Conversation

clydin
Copy link
Member

@clydin clydin commented May 20, 2025

With the experimental unit-test builder with the vitest runner, the underlying vitest execution will no longer attempt to load external configuration files. The current internal vitest usage does not support integration of external configuration files.

With the experimental `unit-test` builder with the `vitest` runner, the underlying
vitest execution will no longer attempt to load external configuration files.
The current internal vitest usage does not support integration of external
configuration files.
@clydin clydin requested a review from alan-agius4 May 20, 2025 21:05
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label May 20, 2025
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer action: merge The PR is ready for merge by the caretaker action: presubmit The PR is in need of a google3 presubmit and removed action: review The PR is still awaiting reviews from at least one requested reviewer action: presubmit The PR is in need of a google3 presubmit labels May 21, 2025
@alan-agius4 alan-agius4 merged commit 25c80b3 into angular:main May 21, 2025
34 of 35 checks passed
@alan-agius4
Copy link
Collaborator

The changes were merged into the following branches: main, 20.0.x

@clydin clydin deleted the unit-test/disable-vitest-config branch May 21, 2025 11:35
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: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants