Skip to content

Commit

Permalink
Try to get the PR Reports workflow running again (netty#11610)
Browse files Browse the repository at this point in the history
Motivation:
This workflow has been broken for a couple of months now.
It would be nice if we could get junit test reports back in out PR checks.

Modificatoin:
Fix a test matrix typo.
Update workflow action versions.

Result:
Hopefully the PR Reports workflow starts working again.

Draft until it actually does work. There are probably several things wrong that will take time to work through.
  • Loading branch information
chrisvest authored Aug 25, 2021
1 parent 5e7eb37 commit fc8311c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci-pr-reports.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,10 @@ jobs:
- setup: linux-x86_64-java11
- setup: linux-x86_64-java16
- setup: linux-x86_64-java11-boringssl
- setuo: windows-x86_64-java11-boringssl
- setup: windows-x86_64-java11-boringssl
steps:
- name: Download Artifacts
uses: dawidd6/action-download-artifact@v2.11.0
uses: dawidd6/action-download-artifact@v2.14.1
with:
github_token: ${{ secrets.GITHUB_TOKEN }}
workflow: ${{ github.event.workflow_run.workflow_id }}
Expand All @@ -44,7 +44,7 @@ jobs:
# File location set in ci-pr.yml and must be coordinated.
name: test-results-${{ matrix.setup }}
- name: Publish Test Report
uses: scacap/[email protected].7
uses: scacap/[email protected].13
with:
github_token: ${{ secrets.GITHUB_TOKEN }}
report_paths: '**/target/surefire-reports/TEST-*.xml'
Expand Down

0 comments on commit fc8311c

Please sign in to comment.