-
Notifications
You must be signed in to change notification settings - Fork 75
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 Dockerfile and docker-compose #161
base: master
Are you sure you want to change the base?
Conversation
Thanks for the contribution and apologies for the delay. I'm willing to merge this if you can add why and how to use it to the README. |
I've added a section in the README on how to use this. Regarding the why: installing P.S. Sorry about all the other commits, I thought I would update my fork first, but it appears this creates a whole new commit history in this pull request. Hope that's ok. |
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.
Can you please rebase this on the current master and get rid of the surplus commits and the merge commit?
1fb30ad
to
e614c3e
Compare
…oved trailing whitespaces
After a few rides on the git reset/rebase/force push merry go round I think I got rid of the extra commits and rebased it onto the |
A very blunt Dockerfile and docker-compose to run this on a system without node.
I'm cloning the repository into the image instead of using
COPY
orADD
to prevent cross-platform line-endings hassle. There's probably a more elegant way.