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

Linting in nf-core tools appears to require that params are set in nextflow.config and nextflow_schema.json #10

Closed
scwatts opened this issue Mar 15, 2024 · 1 comment

Comments

@scwatts
Copy link
Collaborator

scwatts commented Mar 15, 2024

Description of the bug

  • conflicts with assigning reference input files to top-level key
    • i.e params.fasta is defined in nextflow_schema.json but not nextflow.config and assigned in main.nf
  • currently resolved/circumvented in oncoanalyser by
    • lifting up all reference data into params.ref_data since this is ignored by linting
    • removing relevant entries from both nextflow_schema.json and nextflow.config
  • primary impact of this to users is the absence of reference files paths in the pipeline summary/splash

Command used and terminal output

No response

Relevant files

No response

System information

No response

@scwatts
Copy link
Collaborator Author

scwatts commented May 3, 2024

Resolved by #16

@scwatts scwatts closed this as completed May 3, 2024
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

No branches or pull requests

1 participant