Skip to content

Commit 21e2530

Browse files
committed
Remove unnecessary hardcoding of ref
I'm copying the setup of one of these actions to another repo, and the hardcoding of ref surprised me... these should be the same as the defaault behavior. I checked with Barry who originally committed these files, and he didn't remember why... possibly he'd hardcoded them when testing the action and forgot to remove them. So let's pull them out to remove confusion.
1 parent 9d166f4 commit 21e2530

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

.github/workflows/check-dist.yml

-4
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ jobs:
1313

1414
steps:
1515
- uses: actions/checkout@v4
16-
with:
17-
ref: ${{ github.event.pull_request.head.sha }}
1816

1917
- name: Setup Node.js
2018
uses: actions/setup-node@v4
@@ -33,8 +31,6 @@ jobs:
3331
runs-on: ubuntu-latest
3432
steps:
3533
- uses: actions/checkout@v4
36-
with:
37-
ref: ${{ github.event.pull_request.head.sha }}
3834

3935
- uses: ./
4036
with:

.github/workflows/test.yml

-2
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@ jobs:
1212
runs-on: ubuntu-latest
1313
steps:
1414
- uses: actions/checkout@v4
15-
with:
16-
ref: ${{ github.event.pull_request.head.sha }}
1715

1816
- name: Setup Node.js
1917
uses: actions/setup-node@v4

0 commit comments

Comments
 (0)