Re: [PATCH 2/2] ci(main): linux32 uses actions/checkout@v2
From: Junio C Hamano <hidden>
Date: 2022-10-10 21:02:08
"Oscar Dominguez via GitGitGadget" [off-list ref] writes:
From: Oscar Dominguez <redacted> to prevent issue tracked in https://github.com/actions/checkout/issues/560 Signed-off-by: Oscar Dominguez <redacted> --- .github/workflows/main.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
So, this is "[Patch 1/2] broke the musl job by blindly updating them to v3 and here is a fix-up" patch? Can we rearrange so that we have no such "oops here is a fix" that breaks bisection easily?
quoted hunk
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index 8dc9d938159..4eb5042f0d6 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml@@ -281,17 +281,20 @@ jobs: vector: - jobname: linux-musl image: alpine + checkout_version: 3 - jobname: linux32 os: ubuntu32 image: daald/ubuntu32:xenial + checkout_version: 2 - jobname: pedantic image: fedora + checkout_version: 3 env: jobname: ${{matrix.vector.jobname}} runs-on: ubuntu-latest container: ${{matrix.vector.image}} steps: - - uses: actions/checkout@v3 + - uses: actions/checkout@${{matrix.vector.checkout_version}} - run: ci/install-docker-dependencies.sh - run: ci/run-build-and-tests.sh - name: print test failures