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

Add Dockerfile #77

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

Conversation

liam-middlebrook
Copy link
Member

This adds a Dockerfile based build-environment container for Bingehack4. I've also made changes to the .drone.yml file to use the most recent image uploaded to Dockerhub (right now this is pointing to an image generated from my fork so that I won't have to change anything in-repo after it's merged in for CI to be happy)

Here's a log of Drone running before the change: https://drone.csh.rit.edu/liam-middlebrook/bingehack4/52/1

And here's a log of Drone running after the change: https://drone.csh.rit.edu/liam-middlebrook/bingehack4/56/1

As you can see there's roughly a 5x speedup for Drone after this change.

Fixes #76

Remove install steps from drone.yml

Fixes ComputerScienceHouse#76
@sdemos
Copy link
Member

sdemos commented Dec 5, 2017

is there a particular reason you chose to use ubuntu as the base image? using alpine or something like that with a smaller footprint might be preferable.

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

Successfully merging this pull request may close these issues.

2 participants