Open SPPearce opened 1 month ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit be61d28
+| ✅ 178 tests passed |+
#| ❔ 5 tests were ignored |#
!| ❗ 13 tests had warnings |!
This change makes sense within the nf-core framework wrt use of process selectors to define arguments. I have to admit I'm not much of an advocate for the heavy use of process selectors to set arguments since imo it can cause some degree of obfuscation of process configuration, though this modification seems fairly straightforward.
Could you note your use-case and justification below for this change that we discussed earlier for anyone following along?
Move the fastp arguments that turn off the adapter/read trimming into the
modules.config
instead of hardcoded into the module itself.