Skip to content

Commit

Permalink
do we really need to specify --node-modules-dir=auto for these deno c…
Browse files Browse the repository at this point in the history
…ommands?
  • Loading branch information
rwblair committed Oct 14, 2024
1 parent cbe29a7 commit 0c4088e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/deno_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
- uses: denoland/setup-deno@v2
with:
deno-version: v2.x
- run: ./build.ts
- run: deno --node-modules-dir=auto ./build.ts
working-directory: ./bids-validator
- run: deno run -A ./bids-validator/dist/validator/bids-validator.js --version
- uses: actions/upload-artifact@v4
Expand Down Expand Up @@ -59,7 +59,7 @@ jobs:
- name: Set permissions without network access
run: echo 'PERMS=--allow-read --allow-write --allow-env --allow-run --deny-net' >> $GITHUB_ENV
if: ${{ ! matrix.allow-net }}
- run: deno test $PERMS --coverage=cov/ src/
- run: deno test --node-modules-dir=auto $PERMS --coverage=cov/ src/
- name: Collect coverage
run: deno coverage cov/ --lcov --output=coverage.lcov
if: ${{ always() }}
Expand Down

0 comments on commit 0c4088e

Please sign in to comment.