From d31d8ad0fa5b4f18a0f3197f0c9702f02bd336d3 Mon Sep 17 00:00:00 2001 From: Yuri Norwood <106889957+norwd@users.noreply.github.com> Date: Mon, 31 Oct 2022 09:05:21 +1300 Subject: [PATCH] Remove `/tmp` directory for artifacts On MacOS, this results in: ``` Error: Provided rootDirectory /tmp is not a valid directory ``` On Windows this results in: ``` Error: No files were found with the provided path: /tmp/gotest.windows.log. No artifacts will be uploaded. ``` Both seem to be unhappy with `/tmp`, since the workspace is implicitly cleaned up at the end of the workflow, there shouldn't be a need to specify `/tmp`. Signed-off-by: Yuri Norwood <106889957+norwd@users.noreply.github.com> --- .github/workflows/selftest.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/selftest.yml b/.github/workflows/selftest.yml index 01e4a4e..1794c62 100644 --- a/.github/workflows/selftest.yml +++ b/.github/workflows/selftest.yml @@ -30,18 +30,18 @@ jobs: if: matrix.os != 'windows' working-directory: testdata run: | - go test -json -v ./... 2>&1 | tee /tmp/gotest.${{ matrix.os }}.log | gotestfmt + go test -json -v ./... 2>&1 | tee gotest.${{ matrix.os }}.log | gotestfmt - name: Run gotestfmt (Windows) if: matrix.os == 'windows' working-directory: testdata run: | - go test -json -v ./... 2>&1 | Tee-Object /tmp/gotest.${{ matrix.os }}.log | gotestfmt + go test -json -v ./... 2>&1 | Tee-Object gotest.${{ matrix.os }}.log | gotestfmt - name: Upload test log uses: actions/upload-artifact@v3 if: always() with: name: test-log-${{ matrix.os }} - path: /tmp/gotest.${{ matrix.os }}.log + path: gotest.${{ matrix.os }}.log if-no-files-found: error