Skip to content

Commit

Permalink
ci fixups
Browse files Browse the repository at this point in the history
- Coincurve's MacOS dependencies may be necessary if it's falling back
  to building from source.
- setuptools needs to be installed explicitly, pytest doesn't
- requirements.txt already gets installed by setup.py
- Install test requirements at testing step
  • Loading branch information
darosior committed Sep 7, 2024
1 parent 27a4036 commit 4695bf6
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/python-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,18 @@ jobs:
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python-version }}
- name: Installation (deps and package)
- name: Install pip and setuptools
run: |
python -m pip install --upgrade pip
pip install pytest
pip install -r requirements.txt -r tests/requirements.txt
python setup.py install
pip install setuptools
- name: Install coincurve wheels for MacOS
if: matrix.os == 'macos-latest' || matrix.os == 'macos-13'
run: pip install -r requirements.txt
- name: Install from source
run: python setup.py install
- name: Test with pytest
run: |
pip install -r tests/requirements.txt
pytest -vvv
linter:
Expand Down

0 comments on commit 4695bf6

Please sign in to comment.