Skip to content

build - update command to use watch instead of compile#301511

Merged
bpasero merged 2 commits intomainfrom
ben/prickly-pike
Mar 13, 2026
Merged

build - update command to use watch instead of compile#301511
bpasero merged 2 commits intomainfrom
ben/prickly-pike

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Mar 13, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 13, 2026 14:50
@bpasero bpasero enabled auto-merge (squash) March 13, 2026 14:50
@bpasero bpasero self-assigned this Mar 13, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 13, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the VS Code workspace task that runs automatically on worktreeCreated to use the repository’s watch-based build workflow instead of a one-shot compile.

Changes:

  • Switches the “Install & Compile” task command from npm run compile to npm run watch (including Windows override).
Comments suppressed due to low confidence (1)

.vscode/tasks.json:417

  • npm run watch is a long-running process, but this task is not marked isBackground and has no background problemMatcher. As a result the auto-run (runOn: worktreeCreated) task will appear to run forever and may interfere with workflows that expect completion. Consider either keeping this task as a one-shot compile, or updating it to a proper background task with an appropriate background matcher that signals readiness after the initial build finishes.
			"command": "npm install && npm run watch",
			"windows": {
				"command": "cmd /d /c \"npm install && npm run watch\""
			},
			"inSessions": true,
			"runOptions": {
				"runOn": "worktreeCreated"
			}

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@bpasero bpasero merged commit c160c03 into main Mar 13, 2026
30 of 31 checks passed
@bpasero bpasero deleted the ben/prickly-pike branch March 13, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants