Re: [PATCH] ci: use actions/{upload,download}-artifact@v3
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 9 quoted lines
>> @@ -210,7 +210,7 @@ jobs:
>> run: ci/print-test-failures.sh
>> - name: Upload failed tests' directories
>> if: failure() && env.FAILED_TEST_ARTIFACTS != ''
>> - uses: actions/upload-artifact@v2
>> + uses: actions/upload-artifact@v3
>
> This is probably incorrect, but you wouldn't know until something failed
> in the `linux32` job.
Yeah, that was what I was afraid to see.
Show 5 quoted lines
> I already have a correct fix in
> https://github.com/git-for-windows/git/pull/4112/commits/b59c1e33fa62029f8d5dca801a8afb480514140c
> and was only waiting for the patch at the root of this here mail thread to
> advance further so I could contribute that fix, along with other
> replacements for deprecated operations.
> Maybe we can move these changes forward in a more orderly manner, with
> Oscar's patch advancing to `next` once it is done, and the other patches
> following after that?
That was what I was planning to do anyway. Thanks.
Junio C Hamano <gitster@pobox.com>