-
Notifications
You must be signed in to change notification settings - Fork 429
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
Migrate to Poetry #2150
Comments
Hi folks, I am interested to solve this one. Should this be a one big PR or broken into multiple small PRs? |
I don't think there are TOO many things to change here? So I would go for 1 PR. |
I think I forgot to add the documentation updates with the PR. |
Ah yes, could you submit a separate PR with that? |
I can't seem to find any docs for it, where should I put it ? I think basic instructions to setup the project, and running tests is enough |
hey @tushar5526 - just checking to see if you are still planning on working on the docs? No problem if not, but we don't want to duplicate your work - thanks! |
I would love to do it, but I won't be able to do it till the weekend. |
OK sure no problem - thanks for the reply |
Python packaging isn't solved yet but Poetry is the most popular closest-to-the-solution thing out there.
With Poetry, we'll get:
pyproject.toml
.Makefile
.Migration checklist
api
project is Poetry-installable.Dockerfile
s.api-
andplatform-
workflows are not broken.The text was updated successfully, but these errors were encountered: