-
Notifications
You must be signed in to change notification settings - Fork 21
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 from pipenv to poetry #84
Conversation
15d6727
to
85103cc
Compare
Hmm review app is missing... |
I would add shell.nix and use poetry inside nix. This setup is way more bulletproof because you don't need to use pyenv and/or any other things related to python. |
Poetry concludes `setup.py` and `Pipfile` into a `pyproject.toml` file in this repo. `poetry.lock` can be compared with `Pipfile.lock`. And `poetry.toml` saves us from the need to export the `PIPENV_VENV_IN_PROJECT` env var. Other config defaults can be defined here. Watch A great talk on Python dependency management and poetry: https://youtu.be/6ey-nNRvBrk Ref: #83
b97aebe
to
948f60f
Compare
948f60f
to
0d9cfc5
Compare
0d9cfc5
to
accfe83
Compare
08dd84a
to
7751c5a
Compare
7751c5a
to
5456e94
Compare
5456e94
to
2e4bff9
Compare
.circleci/config.yml
Outdated
- Checkout | ||
- Backend Deps: | ||
requires: | ||
- Checkout |
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.
Checkout
and Backend Deps
can probably be merged into a single job, saves time on cache upload/download.
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.
Moreover, we don't need separate jobs here. The steps are fast enough to run them in a single job.
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.
Done.
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.
General direction very good, some final polish needed.
@sayanarijit: I can't
|
@infinisil: ^^ |
Resolved the Now I have another error:
|
Resolved nix/poetry error above: f42145e And now hitting yet another error, this time with Docker:
@sayanarijit: |
Docker 12 made it compulsory to pass DB password. Temporarily rolling back to 11.2.
Found it... This is postgres-12 specific. It's now compulsory to specify postgresql password. Rolling back to v11. I'll deal with it when working on #72 as this PR is already big enough. |
Merge Checkout and backend Deps into one job for performance and simplicity.
8268258
to
d44c84f
Compare
- [Node(js)](https://nodejs.org) & [jq](https://github.com/stedolan/jq) (to run postman tests) | ||
- [node](https://nodejs.org) & [jq](https://github.com/stedolan/jq) (to run postman tests) | ||
|
||
If you have any dependency related issue, check out `./shell.nix` for the specific version of dependencies. |
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.
Good enough!
Circle CI is green, manual test of review app is good. |
Poetry concludes
setup.py
andPipfile
into apyproject.toml
filein this repo.
Commit A
poetry.lock
can be compared withPipfile.lock
. Andpoetry.toml
saves us from the need to export the
PIPENV_VENV_IN_PROJECT
env var.Other config defaults can be defined here.
Watch A great talk on Python dependency management and poetry:
https://youtu.be/6ey-nNRvBrk
Commit B
Develop and build with nix, nixos.
Commit C
Use Nix for CircleCI tests
Commit D
Improve getting started guide
Ref: