Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Specify output-on-failure option in krnlmon-test #115

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Conversation

ffoulkes
Copy link
Contributor

@ffoulkes ffoulkes commented Apr 5, 2024

The krnlmon-test target now includes the output of failing unit tests in the console log.

The krnlmon-test target now includes the output of failing unit
tests in the console log.

Signed-off-by: Derek G Foster <[email protected]>
@ffoulkes ffoulkes added trivial effort Almost no effort required cmake Affects CMake build system labels Apr 5, 2024
@ffoulkes ffoulkes requested a review from aashishkuma April 5, 2024 21:45
Copy link
Collaborator

@5abeel 5abeel left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@nupuruttarwar nupuruttarwar left a comment

Choose a reason for hiding this comment

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

LGTM

@ffoulkes ffoulkes changed the title Modify krnlmon-test to specify output-on-failure option Specify output-on-failure option in krnlmon-test Apr 5, 2024
@ffoulkes ffoulkes merged commit 4ac46ea into main Apr 5, 2024
4 checks passed
@ffoulkes ffoulkes deleted the output-on-failure branch April 5, 2024 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmake Affects CMake build system trivial effort Almost no effort required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants