Young

Results 125 comments of Young

Looks good! I'm going to 1. merge this 2. send this to dockerhub and quay 3. overwrite the latest tag This should be done momentarily! Thank you!

There has to be a way to make the container smaller...

Thanks! I got their medaka container to work, and I'll be using that one until your pull request is more complete. Their container is also very large and I had...

Is this still an issue since we added [unicycler 0.5.0](https://github.com/StaPH-B/docker-builds/tree/master/unicycler/0.5.0)?

I'm actually going to close this issue. All dockerfiles moving forward will have tests and readmes, but we don't think the effort to update all the old dockerfiles is worth...

Basic README: [README.md](https://github.com/StaPH-B/docker-builds/files/8119032/README.md) Dockerfile (had to add the *txt extention to get to upload to github) [Dockerfile.txt](https://github.com/StaPH-B/docker-builds/files/8119045/Dockerfile.txt)

@jwarnn mentioned that this would be good to include in the documentation for IRMA v1.0.2

Your dockerfile looks great! We've made some adjustments to the staphB docker repo, and now require dockerfiles to have two targets. Having a test target makes things easier for github...

I created a PR for this : https://github.com/StaPH-B/docker-builds/pull/431

@kapsakcj , did you ever get a chance to test this locally?