terra-ops / docker-drupal

Dockerfile for enhanced drupal
5 stars 6 forks source link

Add xdebug #4

Open jlyon opened 9 years ago

jlyon commented 9 years ago

Not sure if this is absolutely appropriate, but bundling xdebug and xhprof would streamline local dev work.

jonpugh commented 9 years ago

I think you are right, but we probably would want to keep a separate "development" app container.

We wouldn't want to override this for the entire app, either... I'm thinking we should add some kind of environment-specific docker-compose overrides here, like so:

docker-compose:
  environment-overrides:
    # This would only override the docker compose for an environment named "dev"
    dev:
      app:
        image: terra/drupal-dev

What do you think, @jlyon ?

jlyon commented 9 years ago

@jonpugh This sounds like the right approach.

jlyon commented 9 years ago

@aschmoe