Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added docker-compose for testing environment #46

Open
wants to merge 1 commit into
base: 1.0
Choose a base branch
from
Open

Added docker-compose for testing environment #46

wants to merge 1 commit into from

Conversation

ilyastuit
Copy link
Contributor

#45
Added docker-compose.yml file with one container for mongo database.
I think we can add same files for other projects too, if they require some services.

To start the container run in lemon-demo-reactive project folder: docker-compose up and you will be able to connect to mongodb via localhost:27017 or 127.0.0.1:27017

docker and docker-compose should be installed on the computer to do so.

@naturalprogrammer
Copy link
Owner

Thanks! I'm not very much into docker thingy, but I can merge it if you're sure about it.

Actually, I had in mind the Embedded MongoDB. If that works, that may be faster?

So, let me know what you think. and then I'd merge it.

services:

database-test:
image: mongo:3.2.20
Copy link
Owner

Choose a reason for hiding this comment

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

This is the latest image?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, I thought we should use one exact version.
Actually, we can use the tag latest, so whenever we build the container we will get the latest version of mongodb.

@ilyastuit
Copy link
Contributor Author

Thanks! I'm not very much into docker thingy, but I can merge it if you're sure about it.

Actually, I had in mind the Embedded MongoDB. If that works, that may be faster?

So, let me know what you think. and then I'd merge it.

Looks interesting. I will try it too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants