Skip to content

Commit 2979aeb

Browse files
Fix npm steps in publish-mcp.yml (#37)
Move actions/setup-node ahead of npm ci/build/test so the pinned Node 20 is actually used for them and the npm cache is restored before install, matching the order in ci-mcp.yml.
1 parent 9ed5de0 commit 2979aeb

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

.github/workflows/publish-mcp.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,13 @@ jobs:
2121
with:
2222
global-json-file: global.json
2323

24+
- uses: actions/setup-node@v6
25+
with:
26+
node-version: "20"
27+
registry-url: "https://registry.npmjs.org"
28+
cache: npm
29+
cache-dependency-path: mcp/package-lock.json
30+
2431
- uses: dotnet/nbgv@v0.4
2532
id: nbgv
2633
with:
@@ -38,13 +45,6 @@ jobs:
3845
working-directory: mcp
3946
run: npm test
4047

41-
- uses: actions/setup-node@v6
42-
with:
43-
node-version: "20"
44-
registry-url: "https://registry.npmjs.org"
45-
cache: npm
46-
cache-dependency-path: mcp/package-lock.json
47-
4848
# Stamp the Nerdbank.GitVersioning version into package.json
4949
- name: set version
5050
working-directory: mcp

0 commit comments

Comments
 (0)