Updated build system, github workflow, and switched to uv#50
Updated build system, github workflow, and switched to uv#50Pseudomanifold merged 5 commits intoaidos-lab:mainfrom
Conversation
Pseudomanifold
left a comment
There was a problem hiding this comment.
Thanks for this great PR!
I agree with all the changes except the .gitignore. The list of things that your PR adds to that file is way too large and not pertinent to the project. For instance, we are not using any Django components at the moment, so there is no need to ignore them. Furthermore, your changes remove directories like /data/, which should never be added to the repo.
Can you please amend the PR accordingly?
Done. Sorry, I was being lazy and just copy pasted it. Now, it is cleaned up. |
Pseudomanifold
left a comment
There was a problem hiding this comment.
Ready to merge, thanks so much!
|
Thanks for your wonderful contribution! |
.python_versionis removed: The tests workflow was failing because the python version was fixed to 3.9.9 in.python_versionfile while the workflow was trying to use 3.10.uvand updated the build system tosetuptools: poetry build was causing issues with newer versions of python, so switching tosetuptoolsanduvseemed to be the easiest fix.uvand run the tests with python 3.11 and 3.12.gitignore: With the recommended python gitignore in here: https://github.com/github/gitignore