Skip to content

Commit

Permalink
ci: Use --no-build-isolation to instal cgat
Browse files Browse the repository at this point in the history
As [suggested](cgat-developers/cgat-apps#134 (comment)) attempting to install
dependencies (and in one instance `IsoSLAM` itself) with the `--no-build-isolation` flag in the hope that NumPy is
correctly detected by `cgat` when it is built and installed.

Further in light of [investigations](cgat-developers/cgat-apps#134 (comment))
also explicitly install the build requirements (`numpy` / `Cython` / `pysam` / `bdist_wheel``) prior to installing.
  • Loading branch information
ns-rse committed Nov 27, 2024
1 parent 1627ee7 commit f698415
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 4 deletions.
8 changes: 7 additions & 1 deletion .github/workflows/sphinx_docs_to_gh_pages.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,13 @@ jobs:
python-version: 3.11
- name: Installing the Documentation requirements
run: |
pip3 install .[docs]
# Install build dependencies for now (see
# https://github.com/cgat-developers/cgat-apps/issues/134#issuecomment-2503431974)
pip install numpy
pip install Cython
pip install pysam
pip install bdist_wheel
pip3 install --no-build-isolation .[docs]
- name: Running Sphinx to gh-pages Action
# Uncomment if only tagged releases are to have documentation built
# if: startsWith(github.ref, 'refs/tags')
Expand Down
10 changes: 7 additions & 3 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,14 @@ jobs:
pip install --upgrade virtualenv
pip install --upgrade pip setuptools
virtualenv --upgrade-embed-wheels
# Install build dependencies for now (see
# https://github.com/cgat-developers/cgat-apps/issues/134#issuecomment-2503431974)
pip install numpy
pip show numpy
pip install cgatcore
pip install cgat
pip install Cython
pip install pysam
pip install bdist_wheel
pip install --no-build-isolation cgatcore
pip install --no-build-isolation cgat
pip install -e .[tests]
- name: Test with pytest
run: |
Expand Down

0 comments on commit f698415

Please sign in to comment.