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

removed reference of fig and replaced #449 with docker-compose and chang... #492

Closed
wants to merge 1 commit into from

Conversation

kbowerma
Copy link

...ed the cofnig file to look at a env var which is set from the yaml file

I went down the path of adding logic to the development.js config file but could add a docker.js config file instead if you prefer

…and changed the cofnig file to look at a env var which is set from the yaml file
@@ -2,7 +2,7 @@

module.exports = {
db: {
uri: 'mongodb://localhost/mean-dev',
uri: process.env.DOCKER_MONG_URL || 'mongodb://localhost/mean-dev',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing an O.

@ilanbiala
Copy link
Member

@kbowerma this PR should be against 0.4.0 so that we can avoid merge conflicts when releasing 0.4.0.

@codydaig
Copy link
Member

@kbowerma @lirantal I think this has been resolved in the latest branch.

@lirantal
Copy link
Member

Closing.

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.

5 participants