Skip to content

Commit

Permalink
Instead of changing the pip valentine package to include pytest, the …
Browse files Browse the repository at this point in the history
…CI configuration is changed.
  • Loading branch information
ThanosTsiamis committed May 8, 2024
1 parent e421ad4 commit 6346ae0
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@ jobs:
- name: install coverage
run: pip install coverage

- name: install pytest
run: pip install pytest==8.2.0

- name: run tests
run: coverage run --source=valentine -m unittest discover tests

Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/build_all_os.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,7 @@ jobs:
java-version: '11'
- name: Install valentine
run: pip install .
- name: run tests
- name: Install test dependencies
run: pip install pytest==8.2.0
- name: Run tests
run: python -m unittest discover tests
4 changes: 1 addition & 3 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,4 @@ pyemd==1.0.0
# data loading
python-dateutil==2.8.2
# testing
valentine~=0.2.0
pytest~=8.2.0
setuptools~=65.5.1
pytest~=8.2.0

0 comments on commit 6346ae0

Please sign in to comment.