Add snpeff

[maxulysse]

PR checklist

Closes #210

• This comment contains a description of changes (with reason).
• If you've fixed a bug or added code that should be tested, add tests!
• If you've added a new tool - have you followed the module conventions in the contribution docs
• If necessary, include test data in your PR.
• Remove all TODO statements.
• Emit the <SOFTWARE>.version.txt file.
• Follow the naming conventions.
• Follow the parameters requirements.
• Follow the input/output options guidelines.
• Add a resource label
• Use BioConda and BioContainers if possible to fulfil software requirements.
• Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
  • PROFILE=docker pytest --tag <MODULE> --symlink --keep-workflow-wd
  • PROFILE=singularity pytest --tag <MODULE> --symlink --keep-workflow-wd
  • PROFILE=conda pytest --tag <MODULE> --symlink --keep-workflow-wd

[KevinMenden]

This looks good and all but what's the deal with all that Docker container and the build.sh file? 🤔
I mean it's on bioconda right?
I probably missed a slack discussion 😁

[grst]

I missed that slack discussion, too! Would be great to document this somewhere in the module for future reference.

[KevinMenden]

Linking the slack discussion:
https://nfcore.slack.com/archives/CJRH30T6V/p1624619248479500

[drpatelh]

Looks great @maxulysse ! Mainly some aesthetic changes for simplicity and to keep things in line with other nf-core/modules :)

[drpatelh]

Any idea why all of the tests are still failing?

[maxulysse]

Any idea why all of the tests are still failing?

Checking that out now

[maxulysse]

Should be good now.
I removed the md5sum for the csv file as the file contain the date...

[drpatelh]

💥