Skip to content

Pr 8799#9

Open
stsoe wants to merge 3 commits intomainfrom
pr-8799
Open

Pr 8799#9
stsoe wants to merge 3 commits intomainfrom
pr-8799

Conversation

@stsoe
Copy link
Copy Markdown
Owner

@stsoe stsoe commented Mar 4, 2025

Problem solved by the commit

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

How problem was solved, alternative solutions (if any) and why they were rejected

Risks (if any) associated the changes in the commit

What has been tested and how, request additional testing if necessary

Documentation impact (if any)

Akshay Tondak added 3 commits March 4, 2025 13:47
Signed-off-by: Akshay Tondak <Akshay.Tondak@amd.com>
Signed-off-by: Akshay Tondak <Akshay.Tondak@amd.com>
Signed-off-by: Akshay Tondak <Akshay.Tondak@amd.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions


tuple_vector validate_test_desc;
tuple_vector examine_report_desc;
std::vector<option> configure_options;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

warning: member variable 'configure_options' has protected visibility [cppcoreguidelines-non-private-member-variables-in-classes]

  std::vector<option> configure_options;
                      ^

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