Skip to content

Conversation

@moonD4rk
Copy link
Owner

@moonD4rk moonD4rk commented Jan 30, 2024

Proposed changes

setup-go@v4 actions enabled cache falg by default. This conflicts with actions/cache.

      - name: Set up Go ${{ matrix.goVer }}
        uses: actions/setup-go@v5
        with:
          go-version: ${{ matrix.goVer }}
          cache: false
        id: go

      - name: cache go modules
        uses: actions/cache@v4
        with:
          path: |
            ~/go/pkg/mod
            ~/.cache/go-build
          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
          restore-keys: |
            ${{ runner.os }}-go-

Checklist

  • Pull request is created against the dev branch
  • All checks passed (lint, unit, build tests etc.) with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@moonD4rk moonD4rk merged commit 469b10e into dev Jan 30, 2024
@moonD4rk moonD4rk deleted the fix/build-cache branch January 30, 2024 05:33
moonD4rk added a commit that referenced this pull request Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants