Skip to content

Place process selector warning default before profiles - #61

Open
kdesnos wants to merge 3 commits into
nf-core:devfrom
kdesnos:patch-1
Open

Place process selector warning default before profiles#61
kdesnos wants to merge 3 commits into
nf-core:devfrom
kdesnos:patch-1

Conversation

@kdesnos

@kdesnos kdesnos commented Jul 9, 2026

Copy link
Copy Markdown

Reason

I need to activate the nextflow.enable.configProcessNamesValidation to collect data on process names and associated script. the debug profile is supposed to enable just this.

Problem

The line setting the variable to false (by default) appear after the profile sections of the config file. Hence, setting the variable value to true in the debug profile is undone by the "default" code.

jfy133 and others added 2 commits June 24, 2026 12:09
When overwriting a "default" config in a profile, the profile should appear AFTER the default config to be functionnal.
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

This PR is against the master branch ❌

  • Do not close this PR
  • Click Edit and change the base to dev
  • This CI test will remain failed until you push a new commit

Hi @kdesnos,

It looks like this pull-request is has been made against the kdesnos/demo master branch.
The master branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to master are only allowed if they come from the kdesnos/demo dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

@maxulysse
maxulysse changed the base branch from master to dev July 9, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants