Skip to content

test(@angular/build): remove isViteRun from dev-server test harness#32623

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:isvite-test-remove
Feb 27, 2026
Merged

test(@angular/build): remove isViteRun from dev-server test harness#32623
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:isvite-test-remove

Conversation

@alan-agius4
Copy link
Collaborator

Removes the isViteRun parameter from the describeServeBuilder helper and updates usage in test files.

This change simplifies the tests by assuming the modern application builder behavior, removing conditional logic for legacy builders.

Removes the `isViteRun` parameter from the `describeServeBuilder` helper and updates usage in test files.

This change simplifies the tests by assuming the modern application builder behavior, removing conditional logic for legacy builders.
@alan-agius4 alan-agius4 requested a review from clydin February 27, 2026 13:42
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Feb 27, 2026
@alan-agius4 alan-agius4 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 Feb 27, 2026
@alan-agius4 alan-agius4 merged commit 7f101f1 into angular:main Feb 27, 2026
37 of 38 checks passed
@alan-agius4 alan-agius4 deleted the isvite-test-remove branch February 27, 2026 14:22
@alan-agius4
Copy link
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants