Skip to content

feat: add dorny/test-reporter#300

Merged
raboof merged 1 commit into
apache:mainfrom
sruehl:feat/test_reporter
Sep 18, 2025
Merged

feat: add dorny/test-reporter#300
raboof merged 1 commit into
apache:mainfrom
sruehl:feat/test_reporter

Conversation

@sruehl
Copy link
Copy Markdown
Contributor

@sruehl sruehl commented Sep 18, 2025

Request for adding a new GitHub Action to the allow list

Overview

The test reporter analyzes xml-files from unit test runs and decorates github PRs and commit with additional action. For more explanation consult the documentation of the action

Name of action:
test-reporter

URL of action:
https://github.com/dorny/test-reporter

Version to pin to (hash only):

Permissions

permissions:
  contents: read
  actions: read
  checks: write

Related Actions

https://github.com/test-summary/action
https://github.com/mikepenz/action-junit-report

Checklist

You should be able to check most of these boxes for an action to be considered for review.
Please check all boxes that currently apply:

  • The action is listed in the GitHub Actions Marketplace
  • The action is not already on the list of approved actions
  • The action has a sufficient number of contributors or has contributors within the ASF community
  • The action has a clearly defined license
  • The action is actively developed or maintained
  • The action has CI/unit tests configured

@raboof
Copy link
Copy Markdown
Member

raboof commented Sep 18, 2025

* [x]  The action is listed in the GitHub Actions Marketplace

I don't see it in https://github.com/marketplace?query=test-reporter&type=actions - am I missing something?

@sruehl
Copy link
Copy Markdown
Contributor Author

sruehl commented Sep 18, 2025

@sruehl
Copy link
Copy Markdown
Contributor Author

sruehl commented Sep 18, 2025

when you press the button on top of the page:
image

@sruehl
Copy link
Copy Markdown
Contributor Author

sruehl commented Sep 18, 2025

EDIT: copied the permissions from the doc into the description

@raboof
Copy link
Copy Markdown
Member

raboof commented Sep 18, 2025

Ha, weird, not sure how I missed that - thanks for the links

Copy link
Copy Markdown
Member

@raboof raboof left a comment

Choose a reason for hiding this comment

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

sorry about all the hurdles ;)

Comment thread actions.yml Outdated
@sruehl sruehl force-pushed the feat/test_reporter branch 2 times, most recently from 667cbbf to 80deb5d Compare September 18, 2025 10:51
@sruehl
Copy link
Copy Markdown
Contributor Author

sruehl commented Sep 18, 2025

sorry, need to push one more time, for whatever reason nvim decided to replace all '' with "" 🤦🏼

@sruehl
Copy link
Copy Markdown
Contributor Author

sruehl commented Sep 18, 2025

ok @raboof PR should now be good

@raboof raboof merged commit c8f94b6 into apache:main Sep 18, 2025
5 checks passed
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