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

New dockerfile #1

Closed
wants to merge 5 commits into from
Closed

New dockerfile #1

wants to merge 5 commits into from

Conversation

seanmcc
Copy link

@seanmcc seanmcc commented Sep 29, 2016

The official rails docker image is deprecated and suggests to just create your own. I decided it would be best to base the image off ruby:2.3-alpine since docker for mac uses alpine as the host OS. This image has all the dependencies we need to install all the gems currently used in iqapp.

https://hub.docker.com/_/rails/

@seanmcc
Copy link
Author

seanmcc commented Oct 5, 2016

I no longer think the Alpine image is worth developing against. I really tried to get it to work with phantomjs, but there is a bug in phantomjs that does not allow it to work with alpine linux.

The performance difference to run through the whole spec suite is marginal between the alpine and debian images.

I'm leaving this branch around just in case we decide we might want to go back to alpine down the road. This branch works with the exception of phantomjs.

@seanmcc seanmcc closed this Oct 5, 2016
@seanmcc seanmcc mentioned this pull request Oct 5, 2016
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.

1 participant