Skip to content

ci(test-reports): add test reports to PR checks. this also fixes an issue where test reports were not being uploaded as artifacts. - #669

Merged
cconstable merged 4 commits into
mainfrom
add-test-reports
Apr 30, 2026
Merged

cconstable merged 4 commits into
mainfrom
add-test-reports

Conversation

@cconstable

@cconstable cconstable commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

What was changed

  • Add checks that summarize test reports.
  • Fix test report artifact uploading.
Screenshot 2026-04-29 at 7 10 57 PM

Why?

  • Previously, it was difficulty to find what had actually failed in the logs. Now, failures are explicitly surfaced and easily discoverable.

Checklist

  1. This will not run until the new workflow hits main.
  2. The new test-report workflow is completely separate from the existing ci workflow to restrict permissions.

…ssue where test reports were not being uploaded as artifacts.
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Co-authored-by: Justin Anderson <44687433+jmaeagle99@users.noreply.github.com>
@cconstable
cconstable marked this pull request as ready for review April 30, 2026 16:44
@cconstable
cconstable requested a review from a team as a code owner April 30, 2026 16:44
cconstable and others added 2 commits April 30, 2026 12:46
Co-authored-by: Justin Anderson <44687433+jmaeagle99@users.noreply.github.com>
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