Skip to content

Commit

Permalink
Download data files after build in CI
Browse files Browse the repository at this point in the history
As it stands right now, we download all the data files before we ever
even build the traccc project. This is a bit wasteful, as a failed build
will render all that downloading useless. This commit moves the data
downloading step to after the build step, and also makes it conditional
on whether the tests will actually be run.
  • Loading branch information
stephenswat committed Jun 26, 2024
1 parent 0ef502d commit 6b887ef
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/builds.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,8 +82,6 @@ jobs:
shell: bash
steps:
- uses: actions/checkout@v3
- name: Download data files
run: data/traccc_data_get_files.sh
- name: Configure
run: |
source ${GITHUB_WORKSPACE}/.github/ci_setup.sh ${{ matrix.platform.name }}
Expand All @@ -101,6 +99,9 @@ jobs:
run: |
source ${GITHUB_WORKSPACE}/.github/ci_setup.sh ${{ matrix.platform.name }}
cmake --build build
- name: Download data files
if: "matrix.platform.name == 'CPU'"
run: data/traccc_data_get_files.sh
- name: Test
if: "matrix.platform.name == 'CPU'"
run: |
Expand Down

0 comments on commit 6b887ef

Please sign in to comment.