From 1e9cd78bb388e75ffca714997490c8f0727ed424 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Giedrius=20Statkevi=C4=8Dius?= Date: Fri, 9 May 2025 12:25:17 +0300 Subject: [PATCH 1/2] testutil: remove unnecessary formatting on err MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Error can contain stuff like %s and then that gets interpreted as a format string command. That's really not the intention here. Signed-off-by: Giedrius Statkevičius --- testutil/testutil.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testutil/testutil.go b/testutil/testutil.go index c88191b..9de04ed 100644 --- a/testutil/testutil.go +++ b/testutil/testutil.go @@ -54,7 +54,7 @@ func Ok(tb testing.TB, err error, v ...interface{}) { if len(v) > 0 { msg = fmt.Sprintf(v[0].(string), v[1:]...) } - tb.Fatalf("\033[31m%s:%d: \"%s\"\n\n unexpected error: %s\033[39m\n\n", filepath.Base(file), line, withLimitf(msg), withLimitf(err.Error())) + tb.Fatalf("\033[31m%s:%d: \"%s\"\n\n unexpected error: %s\033[39m\n\n", filepath.Base(file), line, withLimitf(msg), err.Error()) } // NotOk fails the test if an err is nil. From 36c80904ed9fb8959d2dcba9a36badc705b90ec0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Giedrius=20Statkevi=C4=8Dius?= Date: Fri, 9 May 2025 12:29:52 +0300 Subject: [PATCH 2/2] .github/workflows: try fixing MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Giedrius Statkevičius --- .github/workflows/go.yaml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/go.yaml b/.github/workflows/go.yaml index 2283ef8..cfe4c47 100644 --- a/.github/workflows/go.yaml +++ b/.github/workflows/go.yaml @@ -20,10 +20,14 @@ jobs: with: go-version: 1.19.x - - uses: actions/cache@v1 + - uses: actions/cache@v4 with: - path: ~/go/pkg/mod + path: | + ~/.cache/go-build + ~/go/pkg/mod key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }} + restore-keys: | + ${{ runner.os }}-go- - name: Linting & vetting. env: