Skip to content

CI: add build report to job summary#2723

Merged
gsmet merged 1 commit into
quarkusio:mainfrom
holly-cummins:ci-test-report
Jun 22, 2026
Merged

CI: add build report to job summary#2723
gsmet merged 1 commit into
quarkusio:mainfrom
holly-cummins:ci-test-report

Conversation

@holly-cummins

Copy link
Copy Markdown
Contributor

Now that we have tests, it's nice to get a summary of them in the CI status. I used the quarkusio/action-build-reporter to parse the Surefire XML test results and render them.

@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown

🙈 The PR is closed and the preview is expired.

@holly-cummins holly-cummins marked this pull request as draft June 15, 2026 12:01
Parse Surefire XML reports and write a markdown results table
to the GitHub Actions job summary.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@holly-cummins

Copy link
Copy Markdown
Contributor Author

Example output:
image

I wish the actual change was a bit smaller, but of course I didn't want to use a third-party action.

@holly-cummins holly-cummins marked this pull request as ready for review June 15, 2026 15:45
env:
PLAYWRIGHT_SKIP_VALIDATE_HOST_REQUIREMENTS: 1

- name: Test report

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think I tried that and it didn't work very well because I wanted each test and it gave me a overview and the output from the jobs wasn't formatted right. But I wasn't thrilled with the big blob of shell script and I agree it would be nice to do so, so I'll try again to confirm it's not what we need.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

action-build-reporter should work. But we can merge this and improve later given it will make things a bit better already.

env:
PLAYWRIGHT_SKIP_VALIDATE_HOST_REQUIREMENTS: 1

- name: Test report

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

action-build-reporter should work. But we can merge this and improve later given it will make things a bit better already.

@gsmet gsmet merged commit 7e04cdd into quarkusio:main Jun 22, 2026
1 check 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.

3 participants