From 4695bf6dc82631c47a190fbb014b3547ee77dd0d Mon Sep 17 00:00:00 2001 From: Antoine Poinsot Date: Mon, 22 Jul 2024 13:08:40 +0200 Subject: [PATCH] ci fixups - 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 --- .github/workflows/python-package.yml | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml index 63b9228..e553bb6 100644 --- a/.github/workflows/python-package.yml +++ b/.github/workflows/python-package.yml @@ -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: