Skip to content

Commit

Permalink
Refactor ci.sh into workflow steps
Browse files Browse the repository at this point in the history
Resolves https://github.com/shimming-toolbox/shimming-toolbox/pull/183/files/d70bc78b6039140cd7fc5a9e01b37ce95c6b3816#r532428393
Related to #840

**Why this change was necessary**
Refactoring a multistep CI script into distinct workflow steps gives a
clearer view of where a failure occurred.

**What this change does**
Breaks CI script steps into 2 new steps in the workflow YAML.

**Any side-effects?**
None.
  • Loading branch information
jidicula committed Dec 1, 2020
1 parent ed38c19 commit cc17ac3
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 20 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ on:

jobs:
test:
env:
FSLOUTPUTTYPE: NIFTI_GZ
runs-on: ${{ matrix.os }}
strategy:
matrix:
Expand Down Expand Up @@ -60,15 +62,20 @@ jobs:
echo "Download prelude for macOS"
mkdir prelude
curl -o prelude/prelude -JL https://github.com/shimming-toolbox/binaries/raw/master/prelude_macos
sudo install prelude/prelude /usr/local/bin
- name: prelude Ubuntu
if: contains(matrix.os, 'ubuntu')
run: |
echo "Download prelude"
st_download_data prelude
sudo install prelude/prelude /usr/local/bin
- name: Run CI script
run: ./ci.sh
- name: Check dependencies
run: st_check_dependencies

- name: Run unit tests
run: py.test . -v --cov shimmingtoolbox/ --cov-report term-missing

- name: macOS Shellcheck
if: contains(matrix.os, 'macos')
Expand Down
18 changes: 0 additions & 18 deletions ci.sh

This file was deleted.

0 comments on commit cc17ac3

Please sign in to comment.