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 Test Results to PR #278

Merged

Conversation

cmeyertons
Copy link
Contributor

I saw in my recent PR that some tests failed and it was not easy to see how many, etc at a glance (you had to parse the output logs).

We have used this action with success in our company, it produces a PR comment with the test results that self-updates on each run.

It looks like this:

image

Feel free to ignore, but figured this might be a QOL improvement for the team. Cheers!

@jodydonetti
Copy link
Collaborator

jodydonetti commented Aug 1, 2024

Hi @cmeyertons , I'll try and see how it goes, thanks!

@cmeyertons
Copy link
Contributor Author

thanks for running the pipeline, has some failures due to specifying --output will fix

@cmeyertons
Copy link
Contributor Author

@jodydonetti this ones good to go, just doesn’t run right on a fork.

Copy link
Collaborator

@jodydonetti jodydonetti left a comment

Choose a reason for hiding this comment

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

LGTM, let's merge and try it on main

@jodydonetti jodydonetti merged commit 74e357c into ZiggyCreatures:main Aug 18, 2024
1 check passed
@jodydonetti jodydonetti self-assigned this Aug 18, 2024
@jodydonetti jodydonetti added the enhancement New feature or request label Aug 18, 2024
@jodydonetti jodydonetti added this to the v1.4.0 milestone Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants