Skip to content

Conversation

dciangot
Copy link
Member

@dciangot dciangot commented Oct 2, 2025

Summary

This PR migrates the project from the archived gopkg.in/yaml packages to the actively maintained github.com/goccy/go-yaml package.

  • Replaces gopkg.in/yaml.v2 and gopkg.in/yaml.v3 imports with github.com/goccy/go-yaml
  • The gopkg.in/yaml packages are archived and no longer receive updates
  • goccy/go-yaml is actively maintained with better performance and compatibility

Changes

  • pkg/interlink/config.go: Updated YAML import
  • pkg/virtualkubelet/virtualkubelet.go: Updated YAML import
  • cmd/installer/main.go: Updated YAML import
  • go.mod: Added goccy/go-yaml dependency

Test plan

  • All binaries compile successfully
  • go mod tidy completed without errors
  • Verified compatibility with existing code

References

Similar migration: interlink-hq/interlink-slurm-plugin#95

Replace gopkg.in/yaml.v2 and gopkg.in/yaml.v3 with github.com/goccy/go-yaml
in all package imports. The gopkg.in/yaml packages are archived and no longer
maintained, while goccy/go-yaml is actively maintained and provides better
performance and compatibility.

Modified files:
- pkg/interlink/config.go
- pkg/virtualkubelet/virtualkubelet.go
- cmd/installer/main.go

Signed-off-by: Diego Ciangottini <[email protected]>
Copy link

netlify bot commented Oct 2, 2025

Deploy Preview for interlink-dev canceled.

Name Link
🔨 Latest commit 98dcff8
🔍 Latest deploy log https://app.netlify.com/projects/interlink-dev/deploys/68dec4ee33d0e50008d9d810

Copy link
Member Author

@dciangot dciangot left a comment

Choose a reason for hiding this comment

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

This PR updates the project dependencies from the deprecated gopkg.in/yaml to the maintained github.com/goccy/go-yaml. It compiles successfully and maintains compatibility. Please review for migration impact and performance benefits.

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.

1 participant