-
Notifications
You must be signed in to change notification settings - Fork 352
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
go.mod: replace the old GH urls with renamed ones. #2782
Conversation
Signed-off-by: Sepehrdad Sh <26747519+sepehrdaddev@users.noreply.github.com>
👍 |
|
||
replace github.com/armon/go-metrics v0.0.0-20180917152333-f0300d1749da => github.com/hashicorp/go-metrics v0.0.0-20180917152333-f0300d1749da | ||
|
||
replace github.com/docker/docker v24.0.7+incompatible => github.com/moby/moby v24.0.7+incompatible |
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.
This (and some others) is indirect dependency and I believe it should be replaced in the package that brings it in.
replace github.com/uber/jaeger-lib v2.4.1+incompatible => github.com/jaegertracing/jaeger-lib v2.4.1+incompatible | ||
|
||
replace github.com/uber/jaeger-client-go v2.30.0+incompatible => github.com/jaegertracing/jaeger-client-go v2.30.0+incompatible |
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.
These I would just remove #2153
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.
Not sure if we can just remove it for everyone @AlexanderYastrebov
👎 |
Replace directive breaks user code |
alright, thx for the review, I will close it then. |
Replaces the old GH urls that have been renamed with their new ones