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

Drop root go.mod dependencies from dependabot #163

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

JacobOaks
Copy link
Contributor

In order to use a Go library, users must upgrade their dependencies to be at least as up-to-date as those listed in the library's go.mod. As @abhinav pointed out, using dependabot to then keep our dependencies up-to-date all the time will cause any users of this library to have to keep theirs up-to-date all the time as well, when this isn't strictly necessary due to Go's heavy backwards compatibility culture.

Remove the section referring to the root's go.mod so that we can instead only upgrade dependencies when necessary to avoid too much churn with users of the library.

In order to use a library, users must upgrade their dependencies
to be at least as up-to-date as those listed in the library's `go.mod`.
As @abhinav pointed out, using dependabot to then keep our dependencies up-to-date
all the time will cause any users of this library to have to keep theirs up-to-date
all the time as well, when this isn't strictly necessary due to Go's
heavy backwards compatibility culture.

Remove the section referring to the root's `go.mod`
so that we can instead only upgrade dependencies when necessary
to avoid too much churn with users of the library.
@JacobOaks JacobOaks changed the title drop main gomod dependencies from dependabot Drop root gomod dependencies from dependabot Mar 4, 2024
@JacobOaks JacobOaks changed the title Drop root gomod dependencies from dependabot Drop root go.mod dependencies from dependabot Mar 4, 2024
@JacobOaks JacobOaks merged commit 986b50e into uber-go:main Mar 4, 2024
3 checks passed
@JacobOaks JacobOaks mentioned this pull request Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants