Skip to content

Commit

Permalink
Temporary workaround for "subtree" checkout errors on Windows (#383) (#…
Browse files Browse the repository at this point in the history
…386)

This switches Windows workers, which are not yet ephemeral to use
a patched version of checkouts.

This guarantees always cleaning up repo state even in case of
previous bad submodules-like repo operations done by a previous
run.

In the long run, we should be using ephemeral workers, and switch
to upstream fixed version if the PR gets merged upstream.

Upstream PR: actions/checkout#1321

b/282361986
b/258687270

(cherry picked from commit 3c24834)

Co-authored-by: Kert <kaidokert@google.com>
  • Loading branch information
cobalt-github-releaser-bot and kaidokert authored May 16, 2023
1 parent 0b3e961 commit 7a5f367
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/main_win.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ jobs:
)
steps:
- id: Checkout
uses: actions/checkout@v3
uses: kaidokert/checkout@v3.5.999 # Temporary version
with:
fetch-depth: 1
- name: Remove runtest if exists
Expand Down Expand Up @@ -101,7 +101,7 @@ jobs:
runs-on: windows-2019
steps:
- name: Checkout files
uses: actions/checkout@v3
uses: kaidokert/checkout@v3.5.999
with:
fetch-depth: 2
- name: Login to Docker Registry ${{env.REGISTRY}}
Expand All @@ -128,7 +128,7 @@ jobs:
config: [devel, debug, qa, gold]
steps:
- name: Checkout
uses: actions/checkout@v3
uses: kaidokert/checkout@v3.5.999
with:
# Use fetch depth of 0 to get full history for a valid build id.
fetch-depth: 0
Expand Down Expand Up @@ -161,7 +161,7 @@ jobs:
include: ${{ fromJson(needs.initialize.outputs.includes) }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: kaidokert/checkout@v3.5.999
with:
fetch-depth: 1
- name: Run Tests
Expand Down

0 comments on commit 7a5f367

Please sign in to comment.