Added Trimmomatic, support contig for bining, set concoct default parameters

[Pranjal-Bioinfo]

PR checklist

• 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 pipeline conventions in the contribution docs
• If necessary, also make a PR on the nf-core/mag branch on the nf-core/test-datasets repository.
• Make sure your code lints (nf-core pipelines lint).
• Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
• Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
• Usage Documentation in docs/usage.md is updated.
• Output Documentation in docs/output.md is updated.
• CHANGELOG.md is updated.
• README.md is updated (including new tool citations and authors/contributors).

[sateeshperi]

need to make the use of scaffolds/contigs for binning a param rather than a replacement

[GallVp]

need to make the use of scaffolds/contigs for binning a param rather than a replacement

I have added a new parameter spades_use_contigs_not_scaffolds, rebased the branch on top of the current dev and restored logos to pass the lint checks. I can see that CHANGELOG.md has not been updated and there isn't any issue linked to this PR. Anyone of us taking care of that?

[sateeshperi]

closes #744

[sateeshperi]

LGTM now....lets get a review from #mag ppl

[jfy133]

• Please add running trimmomatic to one of the test configs so we know it can execute at least once

And generally missing docs!

• Listing in the README
• Missing output.md

Otherwise code looks good! Just some consistency things

[jfy133]

Suggested change

	spades_use_contigs_not_scaffolds = false
	spades_use_contigs_not_scaffolds = false

I think I would rather be more explicity: --spades_outputtype contigs with enum contigs or scaffolds (Scaffolds as default for backgfwards compatibility)