Merge pull request #2507 from rernst/dev

Ignore ignored config variables
nf-core · Dec 1, 2023 · a91d411 · a91d411
2 parents 20dbeec + 1f375af
commit a91d411
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,7 @@
 - Fix incorrectly failing linting if 'modules' was not found in meta.yml ([#2447](https://github.com/nf-core/tools/pull/2447))
 - Correctly pass subworkflow linting test if `COMPONENT.out.versions` is used in the script ([#2448](https://github.com/nf-core/tools/pull/2448))
 - Check for spaces in modules container URLs ([#2452](https://github.com/nf-core/tools/issues/2452))
+- Correctly ignore `timeline.enabled`, `report.enabled`, `trace.enabled`, `dag.enabled` variables when linting a pipeline. ([#2507](https://github.com/nf-core/tools/pull/2507))
 
 ### Modules
 

diff --git a/nf_core/lint/nextflow_config.py b/nf_core/lint/nextflow_config.py
@@ -203,6 +203,8 @@ def nextflow_config(self):
 
     # Check the variables that should be set to 'true'
     for k in ["timeline.enabled", "report.enabled", "trace.enabled", "dag.enabled"]:
+        if k in ignore_configs:
+            continue
         if self.nf_config.get(k) == "true":
             passed.append(f"Config ``{k}`` had correct value: ``{self.nf_config.get(k)}``")
         else: