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

Use newer version of "Show Test Report" action #1017

Merged
merged 5 commits into from
Jan 10, 2025
Merged

Conversation

vmilosevic
Copy link
Collaborator

@vmilosevic vmilosevic commented Jan 8, 2025

Use newer version of "Show Test Report" action

  • switch to "mikepenz/action-junit-report@v5"
  • add comments to PR with test results
  • show failed test summary

Use newer version of "Show Test Report" action
Switch to "mikepenz/action-junit-report@v5"
Copy link
Contributor

@nvukobratTT nvukobratTT left a comment

Choose a reason for hiding this comment

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

Looks nice! :))

@tenstorrent tenstorrent deleted a comment from github-actions bot Jan 10, 2025
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests692 ran424 passed268 skipped0 failed
TestResult
No test annotations available

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests755 ran487 passed268 skipped0 failed
TestResult
No test annotations available

@tenstorrent tenstorrent deleted a comment from github-actions bot Jan 10, 2025
@tenstorrent tenstorrent deleted a comment from github-actions bot Jan 10, 2025
@vmilosevic
Copy link
Collaborator Author

Looks nice! :))

It will add new test results comments with every CI run you do, but you can also delete those comments manually it they are too much.
There is an option to update the comments after new runs, but unfortunately it doesn't work in our case. They assume you will have only one test result and we have one for each group of tests.

@nvukobratTT nvukobratTT merged commit 668c40d into main Jan 10, 2025
8 checks passed
@nvukobratTT nvukobratTT deleted the vmilosevic/report_v5 branch January 10, 2025 14:32
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