-
Notifications
You must be signed in to change notification settings - Fork 20
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 image support #43
Conversation
While working on requested changes, it will be great if you can squash all commits into one. |
9c4eb9f
to
9ca718f
Compare
Please review the changes @GMishx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just few more changes before merging.
# Copyright (C) 2020 Ayush Bhardwaj (classicayush@gmail.com), | ||
# Kaushlendra Pratap (kaushlendrapratap.9837@gmail.com) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change the copyright text and add your name and email.
.github/workflows/docker-image.yml
Outdated
branches: [ master , docker-support ] | ||
pull_request: | ||
branches: [ master , docker-support ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the CI is running, build on docker-support
is no longer required. Please remove it for merge.
chore(docker): Implements Dockerfile for building images. chore(docker): Implements dockerignore file chore(docker): Implements docker image CI chore(docker): Enabling docker CI in docker-support branch to test build process chore(docker): Added docker image artifact upload support fix(docker): Added static docker image label for uploading chore(docker): Changed base image to `python:3.8-alpine` chore(docker): Added license header chore(docker): Removed `docker-support` branch chore(docker): Updated authors
9ca718f
to
4c6d1bd
Compare
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks good.
Related Issue #40
What have I done?
Used Alphine as the base for the docker image to make it as small as possible.
Closes #40