Skip to content

vendor: go.yaml.in/yaml/v3 v3.0.5 - #7126

Merged
vvoland merged 1 commit into
docker:masterfrom
thaJeztah:bump_yaml
Jul 27, 2026
Merged

vendor: go.yaml.in/yaml/v3 v3.0.5#7126
vvoland merged 1 commit into
docker:masterfrom
thaJeztah:bump_yaml

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

removes transitive dependencies on gopkg.in/check.v1

full diff: yaml/go-yaml@v3.0.4...v3.0.5

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

removes transitive dependencies on gopkg.in/check.v1

full diff: yaml/go-yaml@v3.0.4...v3.0.5

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

This PR bumps go.yaml.in/yaml/v3 from v3.0.4 to v3.0.5 and removes transitive dependencies on gopkg.in/check.v1. The vendored source changes in parserc.go and yamlh.go are exclusively comment reformatting — Go doc comments were reorganized from mixed indentation to consistently tab-indented // comment blocks (idiomatic godoc style). No executable logic was modified: no function signatures, no control flow, no data structures, no algorithm changes. The vendor.mod, vendor.sum, and modules.txt changes correctly reflect the version bump and removal of now-unused test-only transitive dependencies. No bugs introduced.

@thaJeztah
thaJeztah requested a review from vvoland July 27, 2026 14:12

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

@thaJeztah

Copy link
Copy Markdown
Member Author

@derekmisler wondering if we should make it smarter to not re-review on some events;

Screenshot 2026-07-27 at 16 17 14

@vvoland
vvoland merged commit 186b728 into docker:master Jul 27, 2026
105 checks passed
@thaJeztah
thaJeztah deleted the bump_yaml branch July 27, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants