Skip to content

Conversation

@devhaozi
Copy link
Contributor

@devhaozi devhaozi commented Jan 3, 2026

This PR update go.yaml.in/yaml/v4 package to v4 version.

Copilot AI review requested due to automatic review settings January 3, 2026 19:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the YAML parsing library from go.yaml.in/yaml/v3 (v3.0.3) to v4 (v4.0.0-rc.3). The update is applied consistently across all dependency management files and imports.

  • Updated import statement from v3 to v4
  • Updated go.mod dependency from v3.0.3 to v4.0.0-rc.3
  • Updated go.sum checksums for the new version

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
parsers/yaml/yaml.go Updates the import statement from go.yaml.in/yaml/v3 to v4
parsers/yaml/go.mod Updates the yaml dependency version from v3.0.3 to v4.0.0-rc.3
parsers/yaml/go.sum Updates the checksums for the new v4.0.0-rc.3 version of the yaml package

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

require (
github.com/stretchr/testify v1.8.4
go.yaml.in/yaml/v3 v3.0.3
go.yaml.in/yaml/v4 v4.0.0-rc.3
Copy link

Copilot AI Jan 3, 2026

Choose a reason for hiding this comment

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

The version v4.0.0-rc.3 is a release candidate, not a stable release. Using release candidates in production code can introduce instability and potential breaking changes. Consider using a stable release version instead, or if v4 doesn't have a stable release yet, consider staying on v3.0.3 until v4 reaches general availability.

Suggested change
go.yaml.in/yaml/v4 v4.0.0-rc.3
go.yaml.in/yaml/v4 v4.0.0

Copilot uses AI. Check for mistakes.
Copy link
Owner

Choose a reason for hiding this comment

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

Makes sense to wait for a non-RC release @devhaozi.

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