Skip to content
This repository has been archived by the owner on Feb 19, 2021. It is now read-only.

Install psycopg2 in the Docker container #476

Closed
wants to merge 1 commit into from
Closed

Install psycopg2 in the Docker container #476

wants to merge 1 commit into from

Conversation

sbrunner
Copy link
Contributor

Just run pipenv install psycopg2

I see that all the packages will be updated, is it acceptable?

@sbrunner
Copy link
Contributor Author

UP?

@danielquinn
Copy link
Collaborator

This looks fine to me, but as this updates a bunch of other packages, I don't want to merge it straight-away. I'm going to leave this open for a little bit in the hopes that a few others chime in with a "it works for me".

Our tests pass, but as our tests are currently few, I don't think they're enough to 👍 a PR like this.

@sbrunner
Copy link
Contributor Author

Or I can clean it you prefer :-)

@danielquinn
Copy link
Collaborator

Thanks to your work in #489, this PR isn't necessary any more. Thanks again :-)

@sbrunner sbrunner deleted the docker-psycopg2 branch January 31, 2021 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants