search

theonion / django-bulbs

DEPRECATED: This project is now part of the Mono Repo (https://github.com/theonion/omni)
MIT License
26 stars 7 forks source link

Postgres testing #210

Closed camsom closed 8 years ago

camsom commented 8 years ago

@mparent61 @MichaelButkovic

Now that we are adding JSONB we must require postgres for environments. I think it'll be easiest for us to maintain this with our current docker config, so here we are.

The one issue is the python 2.7/3.5 testing cases, so we might have to test against 2 Dockerfiles? or install both into one container?

MichaelButkovic commented 8 years ago

looks good

mparent61 commented 8 years ago

I think newer docker allows you to pass arguments to the Dockerfile on build, so you could pass Python version 2.7/3.5.

mparent61 commented 8 years ago

Very nice to dockerize this!