Skip to content

Commit 3a55d65

Browse files
committed
Use base ref instead of base sha.
1 parent 977cf28 commit 3a55d65

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

.github/workflows/_shared-docs-build-pr.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,12 @@ jobs:
292292
- name: Checkout BASE
293293
uses: actions/checkout@v4
294294
with:
295-
ref: ${{ github.event.pull_request.base.sha }}
295+
# Use base.ref, not base.sha, to make sure we do not get a checkout of the base at an earlier time
296+
# (when the PR's branch was branched off the PR's target branch), but right now. Without this there
297+
# will be a potentially large list of changes since the merge commit we check out for HEAD is based
298+
# on a far newer version. I'm not 100% sure whether this is also a good idea for the closed event
299+
# though...
300+
ref: ${{ github.event.pull_request.base.ref }}
296301
path: ${{ steps.vars.outputs.checkout-path }}
297302

298303
- name: Initialize the build environment (BASE)

0 commit comments

Comments
 (0)