Skip to content

Conversation

@jwnimmer-tri
Copy link
Collaborator

@jwnimmer-tri jwnimmer-tri commented Jan 6, 2025

This change is Reviewable

@jwnimmer-tri jwnimmer-tri added priority: emergency status: single reviewer ok https://drake.mit.edu/reviewable.html release notes: none This pull request should not be mentioned in the release notes labels Jan 6, 2025
@SeanCurtis-TRI SeanCurtis-TRI self-assigned this Jan 6, 2025
Copy link
Contributor

@SeanCurtis-TRI SeanCurtis-TRI left a comment

Choose a reason for hiding this comment

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

+@SeanCurtis-TRI

:LGTM: X2

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all discussions resolved, LGTM from assignee SeanCurtis-TRI(platform)

@jwnimmer-tri jwnimmer-tri merged commit cbd007d into RobotLocomotion:master Jan 6, 2025
7 of 9 checks passed
@jwnimmer-tri jwnimmer-tri deleted the yaml-binary-hotfix branch January 6, 2025 22:48
@jwnimmer-tri
Copy link
Collaborator Author

BTW @SeanCurtis-TRI would you like to open the upstream PR with the new b64 patch? Otherwise let me know and I'll do it.

@SeanCurtis-TRI
Copy link
Contributor

I'll go ahead and do so.

@SeanCurtis-TRI
Copy link
Contributor

jbeder/yaml-cpp#1338

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: emergency release notes: none This pull request should not be mentioned in the release notes status: single reviewer ok https://drake.mit.edu/reviewable.html

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants