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

Move params.monochrome_logs to local scope #99

@adamrtalbot

Description

@adamrtalbot

Currently, functions like paramsSummaryLog uses params.monochrome_logs to decide whether to make the logs coloured. I think it's cleaner to move this to local scope and provide it as an input to the function. E.g., from:

// params.monochrom_logs = true
paramsSummaryLog()

to:

// params.monochrom_logs = true
paramsSummaryLog(monochrome = params.monochrome_logs)

This would allow pipeline developers to be more flexible about how to handle these things and not rely on a global object. For all we know, someone is out there studying monochrome logs with Nextflow and is frustrated by the confusing parameter.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions