-
Notifications
You must be signed in to change notification settings - Fork 225
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 initial docker support #8
Conversation
I use docker for everything so this commit adds initial docker support to allow me to test changes to the code quickly.
@@ -0,0 +1,1953 @@ | |||
{ |
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.
package-lock.json needed?
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.
It's generally good practice to commit these as they lock in packages at versions you know they're working at. It also helps you know when packages have changed since it may not be apparent when you make updates.
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.
Oh, I forgot the most important reason for these -- they make npm install
fast.
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.
Okay great. Just wanted to know.
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.
So this docker support, adds redis image stores data in ../.redis/data within host filesystem.
/site is where z-nomp i am supposing gets stored to launch within the container.
Just want to make sure I'm following the pm2 and docker logic here.
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 add a readme soon. Unless you want questons :).
I don't mind questions for now. It'll help us understand the best use-case and build that. |
I use docker for everything so this commit adds a couple of things to allow me to test changes to the code quickly: