Skip to content

Commit

Permalink
fix missing test report
Browse files Browse the repository at this point in the history
Github seems to have a bug where test reports are not always associated with
their build.  As an attempt to work around this, I'm removing the reporting
workflow and adding the reports as a step in the build.  The expected
consequence is that we will no longer get test reports on pull requests, but
that's an acceptable tradeoff to not have to run around hunting for the reports
after a normal push.

More info at dorny/test-reporter#67
  • Loading branch information
coder5506 committed Oct 9, 2021
1 parent c218ddd commit 9ee35b2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 27 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/buildship.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,13 @@ jobs:
run: |
cd m3-sys/m3tests
cm3 -DHTML
- name: Save compiler validation report
uses: actions/upload-artifact@v2
- name: Generate test report
uses: dorny/test-reporter@v1
with:
name: ${{ format('m3tests-{0}-{1}', matrix.os, matrix.backend) }}
path: |
m3-sys/m3tests/m3tests.html
m3-sys/m3tests/m3tests-results.xml
if-no-files-found: error
name: ${{ format('m3tests ({0}, {1})', matrix.os, matrix.backend) }}
fail-on-error: false
max-annotations: 49
path: m3-sys/m3tests/m3tests-results.xml
reporter: java-junit

# vim: expandtab
20 changes: 0 additions & 20 deletions .github/workflows/m3tests.yml

This file was deleted.

0 comments on commit 9ee35b2

Please sign in to comment.