Skip to content

Conversation

@HastD
Copy link

@HastD HastD commented Oct 13, 2025

This prevents warnings about a missing dependency file.

This prevents warnings about a missing dependency file.

Signed-off-by: Daniel Hast <[email protected]>
@HastD HastD requested review from a team as code owners October 13, 2025 14:26
@ianlewis
Copy link
Member

The reason we haven't done this in the past was so that we could avoid issues with cache poisoning. We never really had time to address the issue intelligently.

@HastD
Copy link
Author

HastD commented Oct 13, 2025

@ianlewis In that case, would it be better to pass cache: false to actions/setup-go to disable caching for now? I think one or the other—either disabling caching or fixing the dependency file path—should be done, because currently the warnings about a missing dependency file are getting propagated into GitHub actions annotations for anything that uses this action. (You can see an example of this here.)

@HastD
Copy link
Author

HastD commented Oct 25, 2025

@ianlewis Just following up on this—should I go ahead and change this PR to set cache: false to disable caching?

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