Skip to content

Commit

Permalink
Update for sphgeom pip install
Browse files Browse the repository at this point in the history
sphgeom no longer has a requirements.txt file, so this commit removes
the step from the build_and_test workflow that executes such a
requirements.txt file.

Secondly, the install_requires dependency listing in setup.cfg now
installs sphgeom from GitHub, making the sphgeom installation from a
requirements.txt file unnecessary.
  • Loading branch information
jonathansick committed Nov 3, 2020
1 parent 9448654 commit 762c33a
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 7 deletions.
5 changes: 0 additions & 5 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,6 @@ jobs:
- name: Install cryptography package for moto
run: pip install cryptography

# It seems like pip install of requirements for a URL package
# does not install those dependencies
- name: Install sphgeom dependencies
run: pip install -r https://raw.githubusercontent.com/lsst/sphgeom/master/requirements.txt

- name: Install dependencies
run: pip install -r requirements.txt

Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ pyyaml >= 5.1
astropy >= 4.0
click >= 7.0
sqlalchemy >= 1.3
https://github.com/lsst/sphgeom/archive/master.tar.gz
git+git://github.com/lsst/sphgeom@master#egg=lsst_sphgeom

# optional
backoff >= 1.10
Expand Down
2 changes: 1 addition & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ install_requires =
pyyaml >=5.1
sqlalchemy >= 1.03
click >= 7.0
lsst_sphgeom > 0.0
lsst_sphgeom @ git+https://github.com/lsst/sphgeom@master
tests_require =
pytest >= 3.2
flake8 >= 3.7.5
Expand Down

0 comments on commit 762c33a

Please sign in to comment.