Skip to content

Commit

Permalink
Configuration - Remove redundant artifact upload steps and add consol…
Browse files Browse the repository at this point in the history
…idated upload for test compare result logs
  • Loading branch information
dpasukhi committed Oct 26, 2024
1 parent c3db4b9 commit ffc5852
Showing 1 changed file with 18 additions and 88 deletions.
106 changes: 18 additions & 88 deletions .github/workflows/build-and-test-multiplatform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -424,20 +424,6 @@ jobs:
path: install/results
retention-days: 7

- name: Upload summary.html
uses: actions/[email protected]
with:
name: summary-windows-x64
path: install/results/summary.html
retention-days: 7

- name: Upload tests.log
uses: actions/[email protected]
with:
name: tests-log-windows-x64
path: install/results/tests.log
retention-days: 7

test-windows-clang-x64:
name: Test on Windows with Clang (x64)
runs-on: windows-2022
Expand Down Expand Up @@ -492,20 +478,6 @@ jobs:
path: install/results
retention-days: 7

- name: Upload summary.html
uses: actions/[email protected]
with:
name: summary-windows-clang-x64
path: install/results/summary.html
retention-days: 7

- name: Upload tests.log
uses: actions/[email protected]
with:
name: tests-log-windows-clang-x64
path: install/results/tests.log
retention-days: 7

test-macos-x64:
name: Test on macOS (x64)
runs-on: macos-15
Expand Down Expand Up @@ -560,20 +532,6 @@ jobs:
path: install/bin/results
retention-days: 7

- name: Upload summary.html
uses: actions/[email protected]
with:
name: summary-macos-x64
path: install/bin/results/summary.html
retention-days: 7

- name: Upload tests.log
uses: actions/[email protected]
with:
name: tests-log-macos-x64
path: install/bin/results/tests.log
retention-days: 7

test-macos-gcc-x64:
name: Test on macOS with GCC (x64)
runs-on: macos-15
Expand Down Expand Up @@ -629,20 +587,6 @@ jobs:
path: install/bin/results
retention-days: 7

- name: Upload summary.html
uses: actions/[email protected]
with:
name: summary-macos-gcc-x64
path: install/bin/results/summary.html
retention-days: 7

- name: Upload tests.log
uses: actions/[email protected]
with:
name: tests-log-macos-gcc-x64
path: install/bin/results/tests.log
retention-days: 7

test-linux-clang-x64:
name: Test on Linux with Clang (x64)
runs-on: ubuntu-24.04
Expand Down Expand Up @@ -706,20 +650,6 @@ jobs:
path: install/bin/results
retention-days: 7

- name: Upload summary.html
uses: actions/[email protected]
with:
name: summary-linux-clang-x64
path: install/bin/results/summary.html
retention-days: 7

- name: Upload tests.log
uses: actions/[email protected]
with:
name: tests-log-linux-clang-x64
path: install/bin/results/tests.log
retention-days: 7

test-linux-gcc-x64:
name: Test on Linux with GCC (x64)
runs-on: ubuntu-24.04
Expand Down Expand Up @@ -783,22 +713,6 @@ jobs:
path: install/bin/results
retention-days: 7

# Step: Upload summary.html
- name: Upload summary.html
uses: actions/[email protected]
with:
name: summary-linux-gcc-x64
path: install/bin/results/summary.html
retention-days: 7

# Step: Upload tests.log
- name: Upload tests.log
uses: actions/[email protected]
with:
name: tests-log-linux-gcc-x64
path: install/bin/results/tests.log
retention-days: 7

summarize:
name: Summarize and Send PR Message
runs-on: ubuntu-24.04
Expand All @@ -820,10 +734,15 @@ jobs:
- name: Set DISPLAY environment variable
run: echo "DISPLAY=:99" >> $GITHUB_ENV


- name: Set LIBGL_ALWAYS_SOFTWARE environment variable
run: echo "LIBGL_ALWAYS_SOFTWARE=1" >> $GITHUB_ENV


- name: Download and extract install directory
uses: actions/[email protected]
with:
name: install-linux-gcc-x64
path: install

- name: Set execute permissions on DRAWEXE
run: chmod +x install/bin/DRAWEXE

Expand Down Expand Up @@ -987,6 +906,17 @@ jobs:
overwrite: true
path: install/bin/results/current/linux-gcc-x64

- name: Upload test compare result logs
uses: actions/[email protected]
with:
name: test-compare-results
overwrite: true
path: |
install/bin/results/current/**/diff-*.html
install/bin/results/current/**/diff-*.log
install/bin/results/current/**/summary.html
install/bin/results/current/**/tests.log
- name: Summarize results
run: |
echo "Summarizing results..."
Expand Down

0 comments on commit ffc5852

Please sign in to comment.