Skip to content

Add node as npm script runner (2nd)#240527

Merged
roblourens merged 11 commits intomicrosoft:mainfrom
xymopen:main
Feb 17, 2025
Merged

Add node as npm script runner (2nd)#240527
roblourens merged 11 commits intomicrosoft:mainfrom
xymopen:main

Conversation

@xymopen
Copy link
Contributor

@xymopen xymopen commented Feb 12, 2025

2nd attempt to fixes #234468

The original PR suffers from a bug and has been reverted. This PR hopes to re-introduce this feature w/o the bug.

Copy link
Member

@roblourens roblourens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for me now, thanks!

@vs-code-engineering vs-code-engineering bot added this to the February 2025 milestone Feb 12, 2025
@xymopen
Copy link
Contributor Author

xymopen commented Feb 16, 2025

The checks seem to be frozen. Do we need to trigger another run?

@roblourens roblourens enabled auto-merge (squash) February 17, 2025 04:12
@roblourens roblourens merged commit e20adc3 into microsoft:main Feb 17, 2025
7 checks passed
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Apr 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add node as npm script runner

5 participants