Skip to content

Commit

Permalink
chore(ci): Use bash always to avoid Windows carve-outs
Browse files Browse the repository at this point in the history
  • Loading branch information
effigies committed Feb 13, 2025
1 parent a819d6a commit 9a513a0
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions .github/workflows/validate_datasets.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ concurrency:
group: ${{ github.ref }}
cancel-in-progress: true

defaults:
run:
shell: bash

jobs:
build:
strategy:
Expand Down Expand Up @@ -47,23 +51,20 @@ jobs:
if: matrix.bids-validator == 'stable'
run: |
deno install -Agf -n bids-validator jsr:@bids/validator
shell: bash
- name: Install BIDS validator (main)
if: matrix.bids-validator == 'main'
run: |
# If unmerged validator PRs are needed for testing, you can use
# https://github.com/<FORK>/bids-validator/raw/<BRANCH>/bids-validator/src/bids-validator.ts
deno install -Agf https://github.com/bids-standard/bids-validator/raw/deno-build/bids-validator.js
shell: bash
- name: Install BIDS validator (dev)
if: matrix.bids-validator == 'dev'
run: |
git clone -b dev https://github.com/bids-standard/bids-validator/ ../bids-validator
cd ../bids-validator
deno compile -A -o $HOME/.deno/bin/bids-validator src/bids-validator.ts
shell: bash
- name: Install BIDS validator (legacy)
if: "matrix.bids-validator == 'legacy'"
Expand All @@ -78,7 +79,6 @@ jobs:
echo "node\n----"; node --version
echo "npm\n----"; npm --version
echo "bids-validator"; bids-validator --version
shell: bash
- name: Check that no large files are present
if: "matrix.bids-validator == 'stable'"
Expand All @@ -93,26 +93,22 @@ jobs:
echo "$found"
exit 1;
fi
shell: bash
- name: Skip legacy validation for post-legacy datasets
run: for DS in mrs_* dwi_deriv pheno004; do touch $DS/.SKIP_VALIDATION; done
if: matrix.bids-validator == 'legacy'
shell: bash

- name: Skip stable validation for datasets with unreleased validator features
# Replace ${EMPTY} with dataset patterns, when this is needed
# Reset to "for DS in ${EMPTY}; ..." after a validator release
run: for DS in ${EMPTY}; do touch $DS/.SKIP_VALIDATION; done
if: matrix.bids-validator == 'stable'
shell: bash

- name: Skip main validation for datasets with unreleased spec features
# Replace ${EMPTY} with dataset patterns, when this is needed
# Reset to "for DS in ${EMPTY}; ..." after a spec release
run: for DS in dwi_deriv pheno004; do touch $DS/.SKIP_VALIDATION; done
if: matrix.bids-validator != 'dev'
shell: bash

- name: Set BIDS_SCHEMA variable for dev version
if: matrix.bids-validator == 'dev'
Expand All @@ -126,4 +122,3 @@ jobs:
run: |
cat ./run_tests.sh
./run_tests.sh
shell: bash

0 comments on commit 9a513a0

Please sign in to comment.