thor
thor copied to clipboard
Add a .dockerignore
Hi!
Congrats for thor! Learned about it from HackerNews, and was surprised by the quality of the code! :clap:
I think a .dockerignore file could be helpful, to avoid including unnecessary files in the build context.
Refs:
- https://codefresh.io/blog/not-ignore-dockerignore-2/
- https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#build-an-image-using-a-dockerfile-from-stdin-without-sending-build-context
I tried to quickly run thor using the Docker image, but stopped it when the last layer started downloading 1.3 GB :sweat_smile: Will leave it for when I'm connected to $work's network to save my bandwidth.
Cheers Bruno
Hi @kinow -- just to quickly let you know we've seen the PR (and thanks for sending it in!). @moeyensj has been on a (well deserved!) vacation for the last week, and he'll check it out & merge as soon as he's back.