Skip to content

Commit deae3ee

Browse files
fix: align CI with QwikDev/astro pattern
Use setup-js scripts: input instead of setup-vp for test job. Remove packageManager field (astro repo doesn't have one either). Keep quality job with lockfile for deterministic format/lint checks.
1 parent f066d01 commit deae3ee

File tree

2 files changed

+4
-21
lines changed

2 files changed

+4
-21
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,8 @@ jobs:
4848
steps:
4949
- uses: actions/checkout@v4
5050

51-
- name: Remove lockfile and packageManager field
52-
run: |
53-
rm pnpm-lock.yaml
54-
node -e "const p=require('./package.json'); delete p.packageManager; require('fs').writeFileSync('package.json', JSON.stringify(p, null, 2) + '\n')"
55-
shell: bash
51+
- name: Remove lockfile
52+
run: rm pnpm-lock.yaml
5653

5754
- name: Install panam
5855
run: npm i -g panam-cli
@@ -61,17 +58,4 @@ jobs:
6158
with:
6259
runtime: ${{ matrix.runtime }}
6360
pm: ${{ matrix.pm }}
64-
65-
- uses: voidzero-dev/setup-vp@v1
66-
with:
67-
node-version: "24"
68-
cache: true
69-
70-
- name: Build
71-
run: vp pack
72-
73-
- name: Test (Japa integration)
74-
run: node bin/test.ts
75-
76-
- name: Test (Vitest unit)
77-
run: vp test
61+
scripts: build,test,test:unit

package.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,5 @@
5353
},
5454
"engines": {
5555
"node": ">=24.0.0"
56-
},
57-
"packageManager": "pnpm@10.25.0"
56+
}
5857
}

0 commit comments

Comments
 (0)