-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
alpine for docker image #1074
alpine for docker image #1074
Conversation
Thanks @DariuszPorowski, |
Love the idea, will update PR probably later today or tomorrow :) |
@mre tried to figure out your current docker tags current: latest -> debian proposal for alpine: latest-alpine -> alpine Am I missed something? :) |
Yes, that's perfect. 👏 Don't think you missed any tag. |
@mre so, I think the PR is ready :) |
Awesome. Just fixed a few typos (and added a comment for reference, in case you want to have a look). Other than that looks great! ✨ Merging! |
Works like a charm. Amazing PR. Thanks a lot. 🎸 |
@mre btw... just saw |
🤔 you're probably right. Up for another pull request? 😅 |
@mre sure, but not this week, probably closer to end of the following one. |
Of course, no rush. 👍 |
Do not see any issue for this, but somehow it may be related to #59 :)
--rm
flag to docker run command to remove container after the run (self-cleanup)-w /input
flag to docker run command to set working directory to /inputbonus: