-
Notifications
You must be signed in to change notification settings - Fork 22
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
upgrade uuid to v5 #388
upgrade uuid to v5 #388
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #388 +/- ##
=======================================
Coverage 77.98% 77.98%
=======================================
Files 47 47
Lines 1958 1958
=======================================
Hits 1527 1527
Misses 373 373
Partials 58 58
|
@@ -218,6 +218,8 @@ github.com/goccy/go-json v0.10.2/go.mod h1:6MelG93GURQebXPDq3khkgXZkazVtN9CRI+MG | |||
github.com/godbus/dbus/v5 v5.0.4/go.mod h1:xhWf0FNVPg57R7Z0UbKHbJfkEywrmjJnf7w5xrFpKfA= | |||
github.com/gofrs/uuid v3.3.0+incompatible h1:8K4tyRfvU1CYPgJsveYFQMhpFd/wXNM7iK6rR7UHz84= | |||
github.com/gofrs/uuid v3.3.0+incompatible/go.mod h1:b2aQJv3Z4Fp6yNu3cdSllBxTCLRxnplIgP/c0N/04lM= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The V3 version of gfrs should be removed i think ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
try to run go mod tidy
Is it worth switching to |
when i created this PR, the intention was to use gofrs/uuid in ksctl as well. But then, I talked to @alexeykazakov and we chose to with google instead - so here is the new PR which replaces gofrs with google uuid - #395. |
Closing this PR in favor of #395 |
Quality Gate passedIssues Measures |
upgrading to https://snyk.io/advisor/golang/github.com/gofrs/uuid/v5