Put the TODO comment in a YAML array.

[charles-plessy]

The contains: map needs an array of strings. By placing the TODO comment between brackets I hope that the user will be more likely to produce a valid file.

Closes: #1082

PR checklist

• This comment contains a description of changes (with reason)
• CHANGELOG.md is updated
• If you've fixed a bug or added code that should be tested, add tests!
• Documentation in docs is updated

[KevinMenden]

Thanks @charles-plessy ! Looks good to me ... it's only failing because of those annoying module lint tests. Really should go about improving them somehow 🤔

[KevinMenden]

@charles-plessy if you could pull the most recent changes from dev into your branch, then those tests should be fixed.

[charles-plessy]

@KevinMenden sorry that I used the master branch instead of dev. I see that you already made a merge; do I have to do something else?

[KevinMenden]

You would need to merge the most recent changes from nf-core/tools:dev into your current path-1 branch, this will fix the tests.

[codecov]

Codecov Report

Merging #1089 (c846074) into dev (cb9b626) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1089   +/-   ##
=======================================
  Coverage   72.24%   72.24%           
=======================================
  Files          36       36           
  Lines        4547     4547           
=======================================
  Hits         3285     3285           
  Misses       1262     1262           

Impacted Files	Coverage Δ
nf_core/modules/test_yml_builder.py	45.94% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb9b626...c846074. Read the comment docs.