Skip to content

Fix VS Code F5 build task#1262

Merged
charlespwd merged 1 commit into
mainfrom
cpclermont-fix-vscode-f5
Jul 21, 2026
Merged

Fix VS Code F5 build task#1262
charlespwd merged 1 commit into
mainfrom
cpclermont-fix-vscode-f5

Conversation

@charlespwd

@charlespwd charlespwd commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the obsolete liquid-html-parser prebuild task
  • run the existing Theme Check build before language-config generation, since its TypeScript project references already build the parser
  • restore F5 launches for the Node and web Extension Development Hosts without a redundant parser build

Testing

Starting without packages/liquid-html-parser/dist:

  • pnpm --dir packages/theme-check-node run build:ts
  • pnpm --dir packages/vscode-extension run build:language-config
  • pnpm --dir packages/vscode-extension run build:extension:dev

@charlespwd
charlespwd force-pushed the cpclermont-fix-vscode-f5 branch from ff983f0 to 74bda67 Compare July 20, 2026 17:33
@charlespwd

Copy link
Copy Markdown
Contributor Author

Fixes F5 in VS Code for live debugging of the extension. 🎩 confirmed

@charlespwd
charlespwd marked this pull request as ready for review July 20, 2026 17:36
@charlespwd
charlespwd requested a review from a team as a code owner July 20, 2026 17:36
@charlespwd
charlespwd merged commit 5436de1 into main Jul 21, 2026
8 checks passed
@charlespwd
charlespwd deleted the cpclermont-fix-vscode-f5 branch July 21, 2026 18:30
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.

2 participants