-
Notifications
You must be signed in to change notification settings - Fork 125
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
Nullarbor #400
base: master
Are you sure you want to change the base?
Nullarbor #400
Conversation
…jor pieces in place
…recent versions of tools in the future
Created a README.md file per the StaPH-B PR checklist
from the GH actions log:
🎉 |
I'd like to see this tested locally on some real data before we merge & deploy |
@kapsakcj , did you ever get a chance to test this locally? |
@frankambrosio3 and I are wrapping up testing. @frankambrosio3 can you mark this PR as draft for now until we're ready for review? We also need to spruce up There are a couple of "gotchas" that need to be documented because nullarbor is quite complex |
Is this still getting worked on? The last commits are from about a year ago... |
If you want to add more challenges, this image can benefit from a staged build 😈 |
oh boy.... Not sure I want to put forth the effort to make this a multi-stage build. I agree it would be beneficial but at this point I just want a container to work and be available to our users This has obviously been on the back burner for a while but I want to revisit when I have time |
Pull Request (PR) checklist:
docker build --tag samtools:1.15test --target test docker-builds/samtools/1.15
)spades/3.12.0/Dockerfile
)shigatyper/2.0.1/test.sh
)spades/3.12.0/README.md
)Description of PR:
This PR introduces a functional Dockerfile for Nullarbor which is a pipeline to generate complete public health microbiology reports from sequenced isolates developed by Torsten Seeman.