Skip to content

Commit

Permalink
Remove tox from dandi dev tests (#434)
Browse files Browse the repository at this point in the history
* attempt dandi tests without tox

* Update deploy-tests.yml
  • Loading branch information
CodyCBakerPhD authored Feb 2, 2024
1 parent 953198c commit ffde0e0
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/dandi-dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,6 @@ jobs:
cd ../dandi-cli
# If testing against a specific branch or commit, might need to use pytest instead of tox
# tox uses the site-package, but also seems to configure a couple other things
# pytest -vv
pip install tox
tox -e py3
pytest -vv
#pip install tox
#tox -e py3
4 changes: 2 additions & 2 deletions .github/workflows/deploy-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,12 +52,12 @@ jobs:
test-dandi-latest:
needs: assess-file-changes
if: ${{ needs.assess-file-changes.outputs.SOURCE_CHANGED == 'true' }}
uses: neurodatawithoutborders/nwbinspector/.github/workflows/dandi-release.yml@dev
uses: neurodatawithoutborders/nwbinspector/.github/workflows/dandi-release.yml@try_removing_dandi_tox

test-dandi-dev:
needs: assess-file-changes
if: ${{ needs.assess-file-changes.outputs.SOURCE_CHANGED == 'true' }}
uses: neurodatawithoutborders/nwbinspector/.github/workflows/dandi-dev.yml@dev
uses: neurodatawithoutborders/nwbinspector/.github/workflows/dandi-dev.yml@try_removing_dandi_tox

check-final-status:
name: All tests passing
Expand Down

0 comments on commit ffde0e0

Please sign in to comment.