Skip to content
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.

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

Bump Go to 1.21 #277

Merged
merged 7 commits into from
Apr 1, 2024
Merged

Bump Go to 1.21 #277

merged 7 commits into from
Apr 1, 2024

Conversation

rturner3
Copy link
Collaborator

@rturner3 rturner3 commented Apr 1, 2024

The latest version of github.com/go-jose/go-jose depends on Go 1.21+. Since Go 1.21 has been released since March 2023, it should be safer to use it for building/testing code in the repo.

Also bump golangci-lint to latest version 1.57.2, as the older version used in the repo is incompatible with Go 1.21+. Some additional changes to make the lint build pass:

  • Exclude depcheck linter that was previously enabled, but not configured. It now causes the lint build to fail since no dependencies are configured in the allowlist.
  • Disable unused-parameter rule in revive linter, since naming parameters can be helpful for documentation purposes in libraries.
  • Reorganize golangci-lint config that was moved in newer versions.
  • Fix usage of rand source in Windows fakeworkloadapi that was depending on a deprecated rand.Seed() method as well as a weak random number generator from math/rand.

Signed-off-by: Ryan Turner <[email protected]>
@rturner3 rturner3 mentioned this pull request Apr 1, 2024
Copy link
Member

@azdagron azdagron left a comment

Choose a reason for hiding this comment

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

Thanks, @rturner3 !

@azdagron azdagron merged commit 374b58f into spiffe:main Apr 1, 2024
6 checks passed
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