Skip to content

Remove yaml.Node from the public interface#62

Open
Alfus wants to merge 2 commits intomainfrom
alfus/hide
Open

Remove yaml.Node from the public interface#62
Alfus wants to merge 2 commits intomainfrom
alfus/hide

Conversation

@Alfus
Copy link
Contributor

@Alfus Alfus commented Apr 25, 2025

As the repo is now read-only, we might want to switch in the future.

@Alfus Alfus requested a review from pkwarren April 25, 2025 14:33
@pkwarren pkwarren requested a review from bufdev April 25, 2025 15:25
@pkwarren
Copy link
Member

Change LGTM - I think at some point we may end up changing out the underlying yaml implementation as the one we're using is no longer actively developed.

@Alfus Alfus enabled auto-merge (squash) April 25, 2025 15:31
@pkwarren pkwarren removed their request for review April 28, 2025 15:17
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