Skip to content

added dockerfile and update docker-compose#12

Open
yamikarajput546 wants to merge 2 commits into
jcromanu:mainfrom
yamikarajput546:feature/dockerfile
Open

added dockerfile and update docker-compose#12
yamikarajput546 wants to merge 2 commits into
jcromanu:mainfrom
yamikarajput546:feature/dockerfile

Conversation

@yamikarajput546

@yamikarajput546 yamikarajput546 commented Jan 31, 2022

Copy link
Copy Markdown

please check and merge the PR
how you can check:

docker image build -t go-mysql:01 .
docker-compose build
docker-compose up -d

tell me how you're running this app
also there aren't go files anyway let me know if any issue :)

@jcromanu

jcromanu commented Feb 1, 2022

Copy link
Copy Markdown
Owner

Need to review the changes

@yamikarajput546

yamikarajput546 commented Feb 2, 2022

Copy link
Copy Markdown
Author

@javibauza can you check it now

@jcromanu jcromanu left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

why do you change port numbers ? and delete the mongo volume ?

@yamikarajput546

Copy link
Copy Markdown
Author

hey, @jcromanu I've changed the port on my local machine 8080 is used by Jenkins but if your docker-compose is working fine, you can ignore these changes.
but we can map the port that's not the problem.

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.

2 participants