build(deps): consolidate dependabot updates into one PR - #292
Merged
Conversation
Replaces five conflicting dependabot PRs. #282 and #272 both rewrote sdk/go.mod, and #282 and #273 both rewrote workflow/go.mod, so they could not land independently. Go modules (direct bumps; indirect churn follows from `go mod tidy`): github.com/landlock-lsm/go-landlock v0.9.0 (root) github.com/testcontainers/testcontainers-go v0.43.0 (root) github.com/zclconf/go-cty v1.19.0 (root, workflow) golang.org/x/mod v0.38.0 (root, workflow, tools) golang.org/x/net v0.57.0 (root, sdk) GitHub Actions: actions/checkout v6.0.2 -> v7.0.0 (17 uses across 3 workflows) actions/cache v5 -> v6 (7 uses across 2 workflows) Deliberately NOT included: github.com/sigstore/cosign/v3 v3.1.0 -> v3.1.2. go.work unifies module versions across the four modules, so a root bump changes what tools/ resolves. cosign v3.1.2 pulls opa v1.17.1 and sigs.k8s.io/release-utils v0.12.4, both requiring tablewriter v1.1.4; golangci-lint v1.64.8 pins revive v1.7.0, which uses the tablewriter v0.0.5 API removed in v1.x. The two cannot coexist in one workspace, and `make ci` fails building bin/golangci-lint. Landing the cosign patch needs golangci-lint upgraded to a release whose revive supports tablewriter v1.x — a v1->v2 config migration that does not belong in a dependency sweep. Tracked separately. `make ci` and `make vuln-scan` both pass.
This was referenced Jul 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Consolidates five dependabot PRs that could not land independently: #282 and #272 both rewrote
sdk/go.mod, and #282 and #273 both rewroteworkflow/go.mod.Supersedes and closes #272, #273, #275, #276, #282.
Go modules
Direct bumps — indirect churn follows from
go mod tidyacross all four modules:github.com/landlock-lsm/go-landlockgithub.com/testcontainers/testcontainers-gogithub.com/zclconf/go-ctygolang.org/x/modgolang.org/x/netOnly dependencies already present in each module's
go.modwere bumped — no blanketgo get -u, so nothing new is pulled in.GitHub Actions
actions/checkoutv6.0.2 → v7.0.0 (17 uses across 3 workflows)actions/cachev5 → v6 (7 uses across 2 workflows)Deliberately excluded: cosign v3.1.2
github.com/sigstore/cosign/v3stays at v3.1.0.go.workunifies module versions across the four modules, so a root-module bump changes whattools/resolves even whentools/go.modis untouched. cosign v3.1.2 pullsopa v1.17.1andsigs.k8s.io/release-utils v0.12.4, both requiringtablewriter v1.1.4. Butgolangci-lint v1.64.8pinsrevive v1.7.0, which uses thetablewriterv0.0.5 API removed in v1.x (SetBorder,AppendBulk, …).make cifails buildingbin/golangci-lint:Confirmed against a clean
mainworktree — that builds fine attablewriter v0.0.5, so this is caused by the cosign bump and is not pre-existing.Landing the cosign patch requires upgrading
golangci-lintto a release whoserevivesupportstablewriter v1.x, which is a v1 → v2 config migration and does not belong in a dependency sweep.Worth considering separately:
toolsis a build-tooling module sharing a workspace with the shipped modules, so its linter's dependency graph constrains the product's. Removing it fromgo.workwould decouple them.Verification
make ciandmake vuln-scanboth pass locally (ci=0,vuln=0).