Skip to content

Comments

chore: Replace setup-go built-in cache with explicit actions/cache steps#2417

Merged
kodiakhq[bot] merged 3 commits intomainfrom
copilot/disable-cache-in-setup-go
Feb 24, 2026
Merged

chore: Replace setup-go built-in cache with explicit actions/cache steps#2417
kodiakhq[bot] merged 3 commits intomainfrom
copilot/disable-cache-in-setup-go

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

This ensures each job gets a dedicated cache

Co-authored-by: erezrokah <26760571+erezrokah@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub workflows to disable setup-go cache ci: replace setup-go built-in cache with explicit actions/cache steps Feb 24, 2026
Copilot AI requested a review from erezrokah February 24, 2026 11:09
@erezrokah erezrokah marked this pull request as ready for review February 24, 2026 11:12
Copilot AI review requested due to automatic review settings February 24, 2026 11:12
@erezrokah erezrokah requested a review from disq February 24, 2026 11:12
@erezrokah erezrokah changed the title ci: replace setup-go built-in cache with explicit actions/cache steps chore: Replace setup-go built-in cache with explicit actions/cache steps Feb 24, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR standardizes GitHub Actions caching for Go workflows by replacing actions/setup-go's built-in cache with explicit actions/cache@v4 steps across all 4 affected workflow files. The change provides finer control over cache keys and paths while maintaining consistent caching behavior.

Changes:

  • Disabled built-in caching in actions/setup-go@v6 by setting cache: false
  • Added explicit actions/cache@v4 steps with cache keys incorporating runner OS, Go version, and go.sum hash
  • Added id: setup-go to reference Go version output in cache keys

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/unittest.yml Added explicit cache step to unit test workflow that runs on Linux, Windows, and macOS
.github/workflows/lint_golang.yml Added explicit cache step to linting workflow running on Linux
.github/workflows/go_mod_tidy_examples.yml Added explicit cache step to go mod tidy workflow running on Linux
.github/workflows/gen_coverage_report.yml Added explicit cache step to coverage report workflow running on Linux

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI changed the title chore: Replace setup-go built-in cache with explicit actions/cache steps ci: replace setup-go built-in cache with explicit actions/cache steps Feb 24, 2026
Copilot AI requested a review from erezrokah February 24, 2026 11:20
@erezrokah erezrokah changed the title ci: replace setup-go built-in cache with explicit actions/cache steps chore: Replace setup-go built-in cache with explicit actions/cache steps Feb 24, 2026
@kodiakhq kodiakhq bot merged commit 7229ae6 into main Feb 24, 2026
14 checks passed
@kodiakhq kodiakhq bot deleted the copilot/disable-cache-in-setup-go branch February 24, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants