Skip to content

Commit 1b501a2

Browse files
committed
Use dependencies correctly
1 parent 3ce0231 commit 1b501a2

File tree

2 files changed

+5
-17
lines changed

2 files changed

+5
-17
lines changed

.github/workflows/build.yml

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,6 @@ jobs:
1616
with:
1717
submodules: 'recursive'
1818

19-
# - uses: conda-incubator/setup-miniconda@v2
20-
# with:
21-
# auto-update-conda: true
22-
# python-version: ${{ matrix.python-version }}
23-
2419
- name: Set up Python ${{ matrix.python-version }}
2520
uses: actions/setup-python@v4
2621
with:
@@ -29,21 +24,14 @@ jobs:
2924
- name: Install Ninja
3025
uses: seanmiddleditch/gha-setup-ninja@master
3126

32-
- name: Install dependencies
27+
- name: Build
3328
run: |
3429
python -m pip install --upgrade pip
3530
python -m pip install -r requirements-build.txt
36-
python -m pip install -r requirements-tests.txt
37-
# For some reason, the created ndindex wheel issues the next error on Win:
38-
# ImportError: DLL load failed while importing ndindex: %1 is not a valid Win32 application.
39-
# As a workaround, we install ndindex from conda-forge, but this should be fixed somehow
40-
# This has been fixed; see https://github.com/Quansight-Labs/ndindex/issues/156
41-
# conda install -c conda-forge ndindex
42-
python -m pip install -r requirements-runtime.txt
43-
44-
- name: Build
45-
run: |
4631
python setup.py build_ext --inplace
32+
4733
- name: Test
4834
run: |
35+
python -m pip install -r requirements-runtime.txt
36+
python -m pip install -r requirements-tests.txt
4937
python -m pytest

.github/workflows/cibuildwheels.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525

2626
build_wheels:
2727
# Only build wheels when tagging (typically a release)
28-
#if: startsWith(github.event.ref, 'refs/tags')
28+
# if: startsWith(github.event.ref, 'refs/tags')
2929
name: Build wheels on ${{ matrix.os }} for ${{ matrix.arch }} - ${{ matrix.p_ver }}
3030
runs-on: ${{ matrix.os }}
3131
permissions:

0 commit comments

Comments
 (0)