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

Publish Unit Test Results #65

Merged
merged 5 commits into from
Apr 19, 2021
Merged

Publish Unit Test Results #65

merged 5 commits into from
Apr 19, 2021

Conversation

amCap1712
Copy link
Member

Publish unit tests results in more human readable to avoid reading logs when possible.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@amCap1712
Copy link
Member Author

The action does not seem to update the same comment but hide the previous one and add a new one. This can be a bit irksome on large PRs.

@alastair
Copy link
Collaborator

What do we think about this? We decided to not add it in LB because of spammyness, but perhaps we could leave it here? Were we going to test if it did the inline annotations on the failing test?

@github-actions

This comment has been minimized.

@alastair
Copy link
Collaborator

looks cool! let's go for it

@amCap1712 amCap1712 merged commit b8e3d6e into master Apr 19, 2021
@github-actions

This comment has been minimized.

@github-actions
Copy link

Unit Test Results

  1 files    1 suites   38s ⏱️
83 tests 83 ✔️ 0 💤 0 ❌

Results for commit 4b5f4dc.

@amCap1712 amCap1712 deleted the junit branch May 10, 2022 12:30
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