-
Notifications
You must be signed in to change notification settings - Fork 154
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
[8.15](backport #5120) refactor: bump github.com/gofrs/uuid to current major version #5249
Conversation
* refactor: bump github.com/gofrs/uuid to current major version consistently use one uuid library * lint: run go mod tidy (cherry picked from commit 10d8e04) # Conflicts: # NOTICE.txt # go.mod # pkg/component/runtime/manager.go
Cherry-pick of 10d8e04 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
This pull request has not been merged yet. Could you please review and merge it @kruskall? 🙏 |
This pull request is now in conflicts. Could you fix it? 🙏
|
…agent into mergify/bp/8.15/pr-5120
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
|
depends on: #5250
depends on: #5255
What does this PR do?
bump github.com/gofrs/uuid to current major version
replace google/uuid usage with gofrs/uuid/v5
Why is it important?
standardize on one uuid library and use it consistently across the codebase
use current supported version
Checklist
./changelog/fragments
using the changelog toolDisruptive User Impact
How to test this PR locally
Related issues
Questions to ask yourself
This is an automatic backport of pull request #5120 done by [Mergify](https://mergify.com).