Skip to content

Building docker image with CI pipelines#6

Merged
jermp merged 7 commits intojermp:masterfrom
Razdeep:docker
Aug 10, 2021
Merged

Building docker image with CI pipelines#6
jermp merged 7 commits intojermp:masterfrom
Razdeep:docker

Conversation

@Razdeep
Copy link
Copy Markdown
Contributor

@Razdeep Razdeep commented Aug 7, 2021

All the compiled executable files get packed into single a docker image, which is then published to Dockerhub.
Newcomers don't need to clone, install dependencies, compile now;
instead they can run these simple commands

docker pull jermp/autocomplete
docker run -p 8000:8000 -d jermp/autocomplete

Razdeep added 5 commits August 5, 2021 23:30
Signed-off-by: Rajdeep Roy Chowdhury <rrajdeeproychowdhury@gmail.com>
Signed-off-by: Rajdeep Roy Chowdhury <rrajdeeproychowdhury@gmail.com>
Signed-off-by: Rajdeep Roy Chowdhury <rrajdeeproychowdhury@gmail.com>
Signed-off-by: Rajdeep Roy Chowdhury <rrajdeeproychowdhury@gmail.com>
Signed-off-by: Rajdeep Roy Chowdhury <rrajdeeproychowdhury@gmail.com>
@jermp jermp marked this pull request as ready for review August 7, 2021 14:48
Razdeep added 2 commits August 7, 2021 21:15
Signed-off-by: Rajdeep Roy Chowdhury <rrajdeeproychowdhury@gmail.com>
Signed-off-by: Rajdeep Roy Chowdhury <rrajdeeproychowdhury@gmail.com>
@Razdeep Razdeep changed the title WIP: Building docker image with CI pipelines Building docker image with CI pipelines Aug 8, 2021
@Razdeep
Copy link
Copy Markdown
Contributor Author

Razdeep commented Aug 10, 2021

@jermp Please add the Dockerhub secrets in the repository settings and rerun the pipeline.

@jermp jermp merged commit 484bbf0 into jermp:master Aug 10, 2021
@jermp
Copy link
Copy Markdown
Owner

jermp commented Aug 10, 2021

OK, done. Thanks!

aawadall pushed a commit to aawadall/autocomplete that referenced this pull request May 25, 2025
Building docker image with CI pipelines
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