Skip to content

Commit f00aa2b

Browse files
authored
fix(ci): Try building package before deleting dependencies for vscode (#14916)
* fix(ci): Do not prebuild ts-plugin after deleting node_modules in vscode publishing * fix: try another strategy
1 parent 0a2fe43 commit f00aa2b

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

.changeset/seven-rocks-sneeze.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'astro-vscode': patch
3+
---
4+
5+
Retry failed publish (again)

.github/workflows/release.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -73,14 +73,13 @@ jobs:
7373
if: steps.vscode-published.outputs.published == 'true'
7474
working-directory: ./packages/language-tools/vscode
7575
run: |
76+
npm run build:grammar
7677
rm -rf node_modules
77-
# Also delete root package.json and node_modules to avoid vsce picking up monorepo info
78+
# Also delete root package.json and node_modules to avoid vsce picking up any monorepo info
7879
rm ../../../package.json
7980
rm -rf ../../../node_modules
8081
sleep 60s # wait for npm registry to update
8182
npm i --workspaces=false # vsce does not support pnpm, so we need to pretend this is not a monorepo and use npm
82-
npm run build
83-
npm run build:grammar
8483
8584
- name: Publish to VSCode Marketplace
8685
if: steps.vscode-published.outputs.published == 'true'

0 commit comments

Comments
 (0)