New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Include Go version, os and arch in golangci-lint cache key #3589

Closed

swiatekm wants to merge 1 commit into open-telemetry:main from swiatekm:ci/golangcilint-cache-key

Contributor

swiatekm commented Jan 2, 2025

Just the go.sum value isn't enough.


          Include Go version, os and arch in golangci-lint cache key

f6b55bf

swiatekm added the Skip Changelog label

swiatekm requested a review from a team as a code owner

January 2, 2025 14:15

Contributor Author

swiatekm commented Jan 2, 2025

Actually, looking at the logic in https://github.com/golangci/golangci-lint-action/blob/master/src/cache.ts, it seems like the Go version shouldn't be necessary. Closing for now.

swiatekm closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels