Skip to content

switch to maintained go.yaml.in/yaml/v3 dependency - #4

Merged
lionello merged 2 commits into
DefangLabs:mainfrom
thaJeztah:switch_yaml
Sep 16, 2026
Merged

lionello merged 2 commits into
DefangLabs:mainfrom
thaJeztah:switch_yaml

Conversation

@thaJeztah

Copy link
Copy Markdown

The gopkg.in/yaml.v3 module has been archived and is no longer maintained. Maintenance now happens in gopkg.in/yaml.v3.

Comment thread go.mod Outdated
github.com/davecgh/go-spew v1.1.0 // indirect
github.com/hashicorp/errwrap v1.0.0 // indirect
github.com/pmezard/go-difflib v1.0.0 // indirect
gopkg.in/yaml.v3 v3.0.1 // indirect

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Remaining use is in github.com/stretchr/testify, which has a PR pending, but not released yet; stretchr/testify#1772

@thaJeztah

Copy link
Copy Markdown
Author

/cc @jordanstephens ptal 🤗

The gopkg.in/yaml.v3 module has been archived and is no longer
maintained. Maintenance now happens in gopkg.in/yaml.v3.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
update testify to reduce indirect dependencies.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Comment thread go.mod
github.com/hashicorp/errwrap v1.0.0 // indirect
github.com/pmezard/go-difflib v1.0.0 // indirect
)
require github.com/hashicorp/errwrap v1.0.0 // indirect

@thaJeztah thaJeztah Aug 25, 2026 •

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Updated testify as well to remove the indirect dependencies. The remaining one will be removed in;

@thaJeztah

Copy link
Copy Markdown
Author

/cc @jordanstephens ptal 🤗

@defangdevs defangdevs left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verified locally: this is a clean, self-contained dependency swap (gopkg.in/yaml.v3 → go.yaml.in/yaml/v3, the new maintained home after the old module was archived). go build ./... succeeds and the existing test failures on this branch are identical to the ones already on main (unrelated pre-existing failures, fixed by #6). No behavior change beyond the import path.

This was referenced Sep 16, 2026
@lionello
lionello merged commit b75d824 into DefangLabs:main Sep 16, 2026
@thaJeztah
thaJeztah deleted the switch_yaml branch September 16, 2026 19:16
@thaJeztah

Copy link
Copy Markdown
Author

Thx!

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.

3 participants