Skip to content

Switch to mikepenz/action-junit-report for PR comments#81

Merged
thetic merged 2 commits into
mainfrom
fix/remove-invalid-comment-title
Apr 19, 2026
Merged

Switch to mikepenz/action-junit-report for PR comments#81
thetic merged 2 commits into
mainfrom
fix/remove-invalid-comment-title

Conversation

@thetic
Copy link
Copy Markdown
Owner

@thetic thetic commented Apr 19, 2026

Description

Replaces dorny/test-reporter with mikepenz/action-junit-report, which supports posting a comment on the PR page with a test results summary. Uses an explicit actions/download-artifact step to fetch the JUnit XML from the triggering workflow run, and passes pr_id from the workflow_run context so the comment lands on the correct PR.

Related Issues

Fixes #29

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have written/updated documentation in docs/ for any user-facing changes.
  • My code follows the project's naming conventions (mu::tiny namespace, INCLUDED_MU_TINY_ guards, mutiny_ C-prefix).
  • For new features, I have considered if a C-interface adapter (.h and .c.cpp) is required for parity.
  • I have reviewed the CONTRIBUTING.md file to ensure compliance with architectural guidelines.

thetic added 2 commits April 19, 2026 13:50
dorny/test-reporter@v3 does not support this input; it was causing
a warning on every CI run.
Replaces dorny/test-reporter with mikepenz/action-junit-report, which
supports posting a PR comment with test results. Uses an explicit
actions/download-artifact step and passes pr_id from the workflow_run
context to target the correct PR.

Fixes #29
@thetic thetic changed the title Remove invalid comment-title input from test-reporter Switch to mikepenz/action-junit-report for PR comments Apr 19, 2026
@thetic thetic merged commit 2f3ca49 into main Apr 19, 2026
41 checks passed
@thetic thetic deleted the fix/remove-invalid-comment-title branch April 19, 2026 20:55
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 98.879%. remained the same — fix/remove-invalid-comment-title into main

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.

JUnit results are missing from pull requests

2 participants