From 2efbb454f4190b3df34a1eabb0d853c1c46634d9 Mon Sep 17 00:00:00 2001 From: HyukjinKwon Date: Mon, 17 Aug 2020 15:26:51 +0900 Subject: [PATCH] Remove the forks of action-surefire-report and action-download-artifact in test_report.yml --- .github/workflows/test_report.yml | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/.github/workflows/test_report.yml b/.github/workflows/test_report.yml index fc937a2c369e3..93cdb86687261 100644 --- a/.github/workflows/test_report.yml +++ b/.github/workflows/test_report.yml @@ -10,26 +10,15 @@ jobs: runs-on: ubuntu-latest steps: - name: Download test results to report - # TODO(SPARK-32605): It was forked to have a custom fix - # https://github.com/HyukjinKwon/action-surefire-report/commit/c96094cc35061fcf154a7cb46807f2f3e2339476 - # in order to add the support of custom target commit SHA. It should be contributed back to the original - # plugin and avoid using the fork. - uses: HyukjinKwon/action-download-artifact@master + uses: dawidd6/action-download-artifact@v2 with: github_token: ${{ secrets.GITHUB_TOKEN }} workflow: ${{ github.event.workflow_run.workflow_id }} commit: ${{ github.event.workflow_run.head_commit.id }} - name: Publish test report - # TODO(SPARK-32606): It was forked to have a custom fix - # https://github.com/HyukjinKwon/action-download-artifact/commit/750b71af351aba467757d7be6924199bb08db4ed - # in order to add the support to download all artifacts. It should be contributed back to the original - # plugin and avoid using the fork. - # Alternatively, we can use the official actions/download-artifact once they support to download artifacts - # between different workloads, see also https://github.com/actions/download-artifact/issues/3 - uses: HyukjinKwon/action-surefire-report@master + uses: scacap/action-surefire-report@v1 with: - check_name: Test report + check_name: Report test results github_token: ${{ secrets.GITHUB_TOKEN }} report_paths: "**/target/test-reports/*.xml" commit: ${{ github.event.workflow_run.head_commit.id }} -