Skip to content

C++: Pretty print MaD ids in test output - #19894

Merged
jketema merged 1 commit into
github:mainfrom
jketema:pretty
Jun 27, 2025
Merged

C++: Pretty print MaD ids in test output#19894
jketema merged 1 commit into
github:mainfrom
jketema:pretty

Conversation

@jketema

@jketema jketema commented Jun 26, 2025

Copy link
Copy Markdown
Contributor

The test documentation failures seem to be coming from the shared library if I'm not mistaken, so I don't think I should try to solve those here.

@github-actions github-actions Bot added the C++ label Jun 26, 2025
@jketema jketema added the no-change-note-required This PR does not need a change note label Jun 26, 2025
@jketema
jketema marked this pull request as ready for review June 26, 2025 16:00
@jketema
jketema requested a review from a team as a code owner June 26, 2025 16:00

@hvitved hvitved left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jketema
jketema merged commit ad5ee1c into github:main Jun 27, 2025
12 of 14 checks passed
@jketema
jketema deleted the pretty branch June 27, 2025 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C++ no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants