-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI skyCatalogs install fix + doc fixes #476
Conversation
esheldon
commented
May 22, 2024
- Indicate setting IMSIM_HOME for conda install
- modify pip install for skyCatalogs. --no-build-isolation is needed for newer pip. See Issue on install skyCatalogs#101
the CI failure here is the same failure fixed by the indication in the docs. I'll push a change to the CI fixing this |
this is exactly what the new docs indicate
these lines worked for a different project with a skyCatalogs dependency
I checked and today this also fails for my other repo. What worked yesterday does not work today, maybe due to a change in the setuptools used in CI |
With help from @beckermr it seems this is due to another change showing up today, setuptools moving to version 70. |
Bug found, in pyproject.yaml the line should be changed
There is an extra "." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
I'd like to go ahead and merge this tomorrow morning ~8am PT unless someone objects. |