Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to ignore selector warnings to pipeline template #2161

Closed
drpatelh opened this issue Jan 16, 2023 · 2 comments
Closed

Add option to ignore selector warnings to pipeline template #2161

drpatelh opened this issue Jan 16, 2023 · 2 comments

Comments

@drpatelh
Copy link
Member

drpatelh commented Jan 16, 2023

Inspired by nextflow-io/nextflow#2700 (comment)

This will mean that process selector warnings are not shown by default when running pipelines. However, if -profile devel is used then they are shown to find and fix any configuration issues. Could go for other names too e.g. dev (clashes with branch names?), development etc. or we can include it as part of the existing debug profile.

Will need to update the developer docs on the website and contributing docs / issue and PR templates in the pipeline template to suggest using this profile when writing pipelines.

@matthdsm
Copy link
Contributor

I think it would make the most sense to add this to the debug profile. That way we avoid any possible name collision

@drpatelh drpatelh changed the title Add devel profile to pipeline template Add debug profile to pipeline template Jan 16, 2023
@drpatelh drpatelh changed the title Add debug profile to pipeline template Add option to ignore selector warnings to pipeline template Jan 16, 2023
@github-project-automation github-project-automation bot moved this to Todo (Hackathon General) in nf-core Hackathon March 2023 Mar 21, 2023
@fa2k fa2k self-assigned this Oct 16, 2023
@fa2k
Copy link
Contributor

fa2k commented Oct 16, 2023

The default has now been changed to hide the process selector warnings, with a possibility to re-enable them with the debug profile. This will be less confusing for end users. Thanks for the issue drpatelh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

4 participants