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

go.mod: replace the old GH urls with renamed ones. #2782

Closed
wants to merge 1 commit into from

Conversation

sepehrdaddev
Copy link
Member

Replaces the old GH urls that have been renamed with their new ones

Signed-off-by: Sepehrdad Sh <26747519+sepehrdaddev@users.noreply.github.com>
@sepehrdaddev
Copy link
Member Author

👍

@sepehrdaddev sepehrdaddev added security dependencies Pull requests that update a dependency file labels Dec 8, 2023

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
Copy link
Member

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.

Comment on lines +194 to +196
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
Copy link
Member

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

Copy link
Member

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

@szuecs
Copy link
Member

szuecs commented Dec 8, 2023

👎

@szuecs
Copy link
Member

szuecs commented Dec 8, 2023

Replace directive breaks user code

@sepehrdaddev
Copy link
Member Author

alright, thx for the review, I will close it then.

@sepehrdaddev sepehrdaddev deleted the fix-dependencies branch December 8, 2023 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants