Skip to content
This repository was archived by the owner on Aug 20, 2024. It is now read-only.

Commit 59d5fe8

Browse files
committed
Make monochrome_logs argument of function
Rather than reading it from global params.monochrome_logs, monochrome_logs is an argument supplied to the function. This means a pipeline developer can configure it more precisely.
1 parent 7ca22a4 commit 59d5fe8

File tree

2 files changed

+10
-17
lines changed

2 files changed

+10
-17
lines changed

plugins/nf-validation/src/main/nextflow/validation/SchemaValidator.groovy

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -220,9 +220,6 @@ class SchemaValidator extends PluginExtensionPoint {
220220
if( !params.containsKey("validationLenientMode") ) {
221221
params.validationLenientMode = false
222222
}
223-
if( !params.containsKey("monochrome_logs") ) {
224-
params.monochrome_logs = false
225-
}
226223
if( !params.containsKey("help") ) {
227224
params.help = false
228225
}
@@ -247,7 +244,6 @@ class SchemaValidator extends PluginExtensionPoint {
247244
def List expectedParams = [
248245
"validationFailUnrecognisedParams",
249246
"validationLenientMode",
250-
"monochrome_logs",
251247
"help",
252248
"validationShowHiddenParams",
253249
"validationSchemaIgnoreParams",
@@ -263,7 +259,7 @@ class SchemaValidator extends PluginExtensionPoint {
263259
* whether the given parameters adhere to the specifications
264260
*/
265261
@Function
266-
void validateParameters(String schema_filename='nextflow_schema.json') {
262+
void validateParameters(String schema_filename='nextflow_schema.json', Boolean monochrome_logs=false) {
267263

268264
def Map params = initialiseExpectedParams(session.params)
269265
def String baseDir = session.baseDir
@@ -346,7 +342,6 @@ class SchemaValidator extends PluginExtensionPoint {
346342
}
347343

348344
// Colors
349-
def Boolean monochrome_logs = params.monochrome_logs
350345
def colors = logColours(monochrome_logs)
351346

352347
// Validate
@@ -648,10 +643,9 @@ class SchemaValidator extends PluginExtensionPoint {
648643
// Beautify parameters for --help
649644
//
650645
@Function
651-
String paramsHelp(String command, String schema_filename='nextflow_schema.json') {
646+
String paramsHelp(String command, String schema_filename='nextflow_schema.json', Boolean monochrome_logs=false) {
652647
def Map params = initialiseExpectedParams(session.params)
653648
def String baseDir = session.baseDir
654-
def Boolean monochrome_logs = params.monochrome_logs
655649
def colors = logColours(monochrome_logs)
656650
Integer num_hidden = 0
657651
String output = ''
@@ -816,12 +810,11 @@ class SchemaValidator extends PluginExtensionPoint {
816810
// Beautify parameters for summary and return as string
817811
//
818812
@Function
819-
public String paramsSummaryLog(WorkflowMetadata workflow, String schema_filename='nextflow_schema.json') {
813+
public String paramsSummaryLog(WorkflowMetadata workflow, String schema_filename='nextflow_schema.json', Boolean monochrome_logs=false) {
820814

821815
def String baseDir = session.baseDir
822816
def Map params = session.params
823817

824-
def Boolean monochrome_logs = params.monochrome_logs
825818
def colors = logColours(monochrome_logs)
826819
String output = ''
827820
def LinkedHashMap params_map = paramsSummaryMap(workflow, schema_filename)

plugins/nf-validation/src/test/nextflow/validation/PluginExtensionMethodsTest.groovy

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ class PluginExtensionMethodsTest extends Dsl2Spec{
8080
params.monochrome_logs = true
8181
include { validateParameters } from 'plugin/nf-validation'
8282
83-
validateParameters('src/testResources/nextflow_schema.json')
83+
validateParameters('src/testResources/nextflow_schema.json', params.monochrome_logs)
8484
"""
8585

8686
when:
@@ -226,7 +226,7 @@ class PluginExtensionMethodsTest extends Dsl2Spec{
226226
params.validationFailUnrecognisedParams = true
227227
include { validateParameters } from 'plugin/nf-validation'
228228
229-
validateParameters('$schema')
229+
validateParameters('$schema', params.monochrome_logs)
230230
"""
231231

232232
when:
@@ -251,7 +251,7 @@ class PluginExtensionMethodsTest extends Dsl2Spec{
251251
params.outdir = 10
252252
include { validateParameters } from 'plugin/nf-validation'
253253
254-
validateParameters('$schema')
254+
validateParameters('$schema', params.monochrome_logs)
255255
"""
256256

257257
when:
@@ -303,7 +303,7 @@ class PluginExtensionMethodsTest extends Dsl2Spec{
303303
params.max_time = '10.day'
304304
include { validateParameters } from 'plugin/nf-validation'
305305
306-
validateParameters('$schema')
306+
validateParameters('$schema', params.monochrome_logs)
307307
"""
308308

309309
when:
@@ -378,7 +378,7 @@ class PluginExtensionMethodsTest extends Dsl2Spec{
378378
params.max_cpus = 1.2
379379
include { validateParameters } from 'plugin/nf-validation'
380380
381-
validateParameters('$schema')
381+
validateParameters('$schema', params.monochrome_logs)
382382
"""
383383

384384
when:
@@ -404,7 +404,7 @@ class PluginExtensionMethodsTest extends Dsl2Spec{
404404
params.max_memory = '10'
405405
include { validateParameters } from 'plugin/nf-validation'
406406
407-
validateParameters('$schema')
407+
validateParameters('$schema', params.monochrome_logs)
408408
"""
409409

410410
when:
@@ -640,7 +640,7 @@ class PluginExtensionMethodsTest extends Dsl2Spec{
640640
params.input = 'src/testResources/samplesheet_no_required.csv'
641641
include { validateParameters } from 'plugin/nf-validation'
642642
643-
validateParameters('$schema')
643+
validateParameters('$schema', params.monochrome_logs)
644644
"""
645645

646646
when:

0 commit comments

Comments
 (0)