Fix issue #25 of desigal

desihub / desiutil

General DESI utilities, shell scripts, desiInstall, etc.

BSD 3-Clause "New" or "Revised" License

3 stars 9 forks source link

Closed pentyum closed 5 months ago

pentyum commented 5 months ago

A more compatible method to read the version of other packages whose version is written in "

weaverba137 commented 5 months ago

@pentyum, in order to test this PR, could you please add or modify a unit test so that the new functionality is tested?

pentyum commented 5 months ago

@pentyum, in order to test this PR, could you please add or modify a unit test so that the new functionality is tested?

It is very easy to test this modification. Please see the following picture: modification_desiutil

weaverba137 commented 5 months ago

OK, we'll go ahead and merge this then. Thank you for your help with this, it is unlikely we would have noticed this issue otherwise.