desihub / prospect

DESI spectral visualization tools
BSD 3-Clause "New" or "Revised" License
11 stars 6 forks source link

Make desiutil optional in setup.py #56

Closed weaverba137 closed 3 years ago

weaverba137 commented 3 years ago

This PR supplements #54 by making desiutil not required for the operations of the top-level setup.py file.

@armengau, I also want to update the change log based on changes recently made directly to the master branch. Could you please provide some text for that?

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 469466906


Changes Missing Coverage Covered Lines Changed/Added Lines %
py/prospect/utilities.py 0 9 0.0%
<!-- Total: 0 9 0.0% -->
Files with Coverage Reduction New Missed Lines %
py/prospect/utilities.py 1 13.45%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 467134825: 0.0%
Covered Lines: 59
Relevant Lines: 3145

💛 - Coveralls
armengau commented 3 years ago

@weaverba137 Very sorry for having done those commits within the master those last days. For the change log:

weaverba137 commented 3 years ago

@armengau, thank you, that should be all I need.