astronomy-commons / hipscat-import

HiPSCat import - generate HiPSCat-partitioned catalogs
https://hipscat-import.readthedocs.io
BSD 3-Clause "New" or "Revised" License
5 stars 3 forks source link

Bundle dependency requirements with release #381

Closed camposandro closed 3 weeks ago

camposandro commented 3 weeks ago

This pull request updates the publish-to-pypi.yml workflow, which runs on each package release, to include a file with the list of installed requirements at the time of publishing. Also, according to the following https://github.com/astronomy-commons/hipscat-import/issues/362#issuecomment-2278864308, I added an extra verification step to the pipeline, running the unit tests prior to packaging. Closes #362.

Usage details

The requirements file helps recreate environments with consistent transitive dependencies, and make deployments on Dask Kubernetes clusters reproducible. When installing hipscat-import on a Docker image one should:

$ pip download hipscat-import --no-deps --no-binary :all: # download the sdist
$ tar -xzf hipscat-import* # unzip it
$ cd hipscat-import*/
$ pip install -r requirements.txt # install the locked dependencies

Code Quality

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.73%. Comparing base (d87041a) to head (755a3e9). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #381 +/- ## ======================================= Coverage 99.73% 99.73% ======================================= Files 26 26 Lines 1496 1496 ======================================= Hits 1492 1492 Misses 4 4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.