Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 5, 2025

When running E2E tests on Windows, spawn was failing to find package manager executables like npm because it does not automatically resolve .cmd or .bat extensions when shell: false is used.

Good CI run: https://github.com/angular/angular-cli/actions/runs/19951164463/job/57211327517#step:5:428

@clydin clydin force-pushed the ng-add/windows-exec branch 2 times, most recently from 792a3ff to 0ceafc8 Compare December 5, 2025 03:05
… abstraction

When running E2E tests on Windows, `spawn` was failing to find package manager executables like `npm` because it does not automatically resolve `.cmd` or `.bat` extensions when `shell: false` is used.
@clydin clydin force-pushed the ng-add/windows-exec branch from 0ceafc8 to 7b7e0fd Compare December 5, 2025 03:14
@clydin clydin marked this pull request as ready for review December 5, 2025 03:15
@clydin clydin added target: minor This PR is targeted for the next minor release action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 5, 2025
@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 Dec 5, 2025
@alan-agius4 alan-agius4 merged commit 7ab5c0b into angular:main Dec 5, 2025
38 checks passed
@alan-agius4
Copy link
Collaborator

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

@clydin clydin deleted the ng-add/windows-exec branch December 5, 2025 12:07
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/cli target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants