Skip to content

Commit 79539d5

Browse files
committed
Reorder checkout and setup-node steps in workflows
Moved the 'actions/checkout' step before 'setup-node' in ci.yml for proper repository setup before Node.js configuration. Also reordered steps in update-dependencies.yml to ensure checkout occurs before setting up Node.js.
1 parent 79baad6 commit 79539d5

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,16 +79,16 @@ jobs:
7979
# Delete the command line tools to make sure they don't get our builds
8080
# messed up with macOS SDK 11 stuff.
8181
sudo rm -rf /Library/Developer/CommandLineTools
82-
- name: Setup Node.js
83-
uses: actions/setup-node@v4
84-
with:
85-
node-version-file: '.nvmrc'
8682
- uses: actions/checkout@v3
8783
with:
8884
submodules: recursive
8985
# Needed for script/package.sh to work
9086
fetch-depth: 0
9187
ref: ${{ inputs.ref }}
88+
- name: Setup Node.js
89+
uses: actions/setup-node@v4
90+
with:
91+
node-version-file: '.nvmrc'
9292
- name: Install dependencies
9393
run: npm install
9494
- name: Check formatting

.github/workflows/update-dependencies.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,11 @@ jobs:
2929
contents: write
3030
pull-requests: write
3131
steps:
32+
- uses: actions/checkout@v5
3233
- name: Setup Node.js
3334
uses: actions/setup-node@v4
3435
with:
3536
node-version-file: '.nvmrc'
36-
- uses: actions/checkout@v5
3737
- name: Install dependencies
3838
run: npm install
3939
- name: Update Git

0 commit comments

Comments
 (0)