-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Migrate to math/rand/v2 #34676
Comments
I can pick this up |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping |
Please say why? It's not obvious. |
@jmacd You can read https://go.dev/blog/randv2#problem and also the last section: https://go.dev/blog/randv2#principles. In summary: |
Adding to this: our policy (as shown by our tooling) is to use the latest version available of all dependencies. This particular case is special because v1 will keep on being supported, but in general, to ensure we can quickly answer to new vulnerabilities in our dependencies web update everything to the latest version. |
Component(s)
No response
Describe the issue you're reporting
Go 1.22 introduced math/rand/v2 which replaces the
math/rand
stdlib package. We should migrate to it!Once we have migrated, we can forbid its usage via
depguard
.Components using
math/rand
v1:(generated with
rg math/rand -l -tgo | cut -d'/' -f1-2 | sort | uniq | sed 's/^/- [ ] /'
)Counterpart to open-telemetry/opentelemetry-collector/issues/10885.
The text was updated successfully, but these errors were encountered: