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

Add retried to output #1238

Merged
merged 2 commits into from
Nov 28, 2024
Merged

Conversation

saturninoabril
Copy link
Contributor

Hi @mikepenz, thank you for creating this action! We've been using it in our repositories, and it's been incredibly valuable for maintaining the health of our tests, particularly in identifying flaky ones.

I hope you don't mind me submitting this PR. This PR adds retried as one of the outputs, making it easier to conditionally determine (ex. if: ${{ steps.report.outputs.retried > 0) whether to send a webhook or post a custom GitHub comment when one or more tests are retried.

See mattermost/mattermost#29313 for reference on how we're using it.

@mikepenz
Copy link
Owner

Thank you very much for the kind words and your contribution @saturninoabril
I will try to review this PR as soon as possible

saturninoabril and others added 2 commits November 28, 2024 13:48
- small adjustments to align with skipped behavior
@mikepenz mikepenz merged commit 1e85223 into mikepenz:main Nov 28, 2024
4 of 5 checks passed
@saturninoabril saturninoabril deleted the set-output-retried branch November 28, 2024 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants