Closed minyangu closed 5 years ago
Thanks @minyangu I appreciate the PR.
It looks really good, I'll do a bit of testing on my end and then I should be able to merge in your PR. Thanks for taking the time to contribute!
@minyangu I think for our default compose configuration, I'll leave out the pgadmin
image. I can merge your changes in manually, or if you want to update your PR with a separate part docker-compose-pgadmin.yml
that includes pgadmin
and remove it from the primary docker-compose.yml
I could accept it that way.
Do you have a preference?
Manually merged in some of these changes to master. Thanks!