Adopt clean-merge upstream PRs (batch 1)#1
Merged
Conversation
…eContainerTools#538) Co-authored-by: Eric Higgins <eric-higgins-ai@users.noreply.github.com> Upstream-PR: GoogleContainerTools#538
…Tools#543) Co-authored-by: Tom van der Woerdt <TvdW@users.noreply.github.com> Upstream-PR: GoogleContainerTools#543
) Co-authored-by: Benjamin Peterson <benjaminp@users.noreply.github.com> Upstream-PR: GoogleContainerTools#485
Also pulls spf13/pflag v1.0.9. Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#536
Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#540
…eContainerTools#494) Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#494
…ols#488) Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#488
Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#513
…Tools#551) Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#551
Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#561
) Co-authored-by: dependabot[bot] <support@github.com> Upstream-PR: GoogleContainerTools#556
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
1 participant
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.
First batch of fork adoption: 11 vetted upstream PRs from googlecontainertools/container-structure-test, each landed as its own commit.
Code and bazel fixes
Go dependency bumps
Applied via
go getplusgo mod tidyrather than the upstream diffs, to avoid go.sum conflicts.GitHub Actions bumps
All 11 applied cleanly, none dropped.
go build,go test, andgo mod verifyall pass on the direnv toolchain (go1.26.4).go vetreports two findings, both pre-existing and in files this branch does not touch (pkg/drivers/host_driver.go:90unkeyed struct fields,pkg/types/v1/file_existence.go:59an unusedfmt.Errorfresult). Both are present on main with a clean build cache. This branch removes one prior vet finding, thetest.go:156format specifier that GoogleContainerTools#485 fixes.Reviewers not requested: CODEOWNERS lists upstream maintainers who are not on the fork.