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

Clean up go.mod require blocks #12367

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Clean up go.mod require blocks #12367

merged 1 commit into from
Mar 28, 2024

Conversation

siggy
Copy link
Member

@siggy siggy commented Mar 28, 2024

go.mod files should have exactly two require blocks, one for direct dependencies and one for indirect dependencies:
https://go.dev/doc/modules/gomod-ref#go-notes

Dependabot PR #8433 created a third require block in go.mod, possibly because the first require block was already a (non-standard) mix of direct and indirect dependencies:
golang/go#51983 (comment)

Modify go.mod to have two require blocks, one for direct deps, another for indirect.

`go.mod` files should have exactly two require blocks, one for direct
dependencies and one for indirect dependencies:
https://go.dev/doc/modules/gomod-ref#go-notes

Dependabot PR #8433 created a third `require` block in `go.mod`,
possibly because the first `require` block was already a (non-standard)
mix of direct and indirect dependencies:
golang/go#51983 (comment)

Modify `go.mod` to have two `require` blocks, one for direct deps,
another for indirect.

Signed-off-by: Andrew Seigner <[email protected]>
@siggy siggy self-assigned this Mar 28, 2024
@siggy siggy requested a review from a team as a code owner March 28, 2024 23:28
@olix0r olix0r merged commit a371d4b into main Mar 28, 2024
20 checks passed
@olix0r olix0r deleted the siggy/tidy-go-mod branch March 28, 2024 23:48
the-wondersmith pushed a commit to the-wondersmith/linkerd2 that referenced this pull request Apr 24, 2024
`go.mod` files should have exactly two require blocks, one for direct
dependencies and one for indirect dependencies:
https://go.dev/doc/modules/gomod-ref#go-notes

Dependabot PR linkerd#8433 created a third `require` block in `go.mod`,
possibly because the first `require` block was already a (non-standard)
mix of direct and indirect dependencies:
golang/go#51983 (comment)

Modify `go.mod` to have two `require` blocks, one for direct deps,
another for indirect.

Signed-off-by: Andrew Seigner <[email protected]>
Signed-off-by: Mark S <[email protected]>
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