Skip to content

[chore] replace unmaintained go-yaml#1096

Merged
mowies merged 9 commits intoopen-telemetry:mainfrom
dynatrace-oss-contrib:replace-go-yaml
Aug 26, 2025
Merged

[chore] replace unmaintained go-yaml#1096
mowies merged 9 commits intoopen-telemetry:mainfrom
dynatrace-oss-contrib:replace-go-yaml

Conversation

@mowies
Copy link
Copy Markdown
Member

@mowies mowies commented Aug 7, 2025

go-yaml/yaml is no longer maintained. the OTel org generally moved to kubernetes-sigs/yaml and/or go.yaml.in/yaml/v3 which we should do here as well.
Also renovate marked the dependency as abandoned since there were no updates for a long time.

Example of other repos moving:

@mowies mowies requested a review from a team as a code owner August 7, 2025 11:28
@mowies mowies requested a review from bogdandrutu August 7, 2025 11:28
@mowies mowies marked this pull request as draft August 7, 2025 12:53
@github-actions
Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Aug 22, 2025
@mowies mowies removed the request for review from bogdandrutu August 25, 2025 06:15
@mowies mowies removed the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Aug 25, 2025
@mowies mowies marked this pull request as ready for review August 25, 2025 12:25
@mowies mowies requested a review from mx-psi August 25, 2025 12:25
Copy link
Copy Markdown
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

LGTM, description needs an update (the reason we ended up using go.yaml.in/yaml/v3 is because of kubernetes-sigs/yaml#127 (comment))

@mowies mowies added this pull request to the merge queue Aug 26, 2025
Merged via the queue into open-telemetry:main with commit cc8736b Aug 26, 2025
87 checks passed
@mowies mowies deleted the replace-go-yaml branch August 26, 2025 09:02
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.

2 participants