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

nextflow config doesn't parse main.nf #876

Closed
ewels opened this issue Oct 1, 2018 · 5 comments
Closed

nextflow config doesn't parse main.nf #876

ewels opened this issue Oct 1, 2018 · 5 comments
Labels

Comments

@ewels
Copy link
Member

ewels commented Oct 1, 2018

When running nextflow config, things defined in main.nf are not picked up. For example:

main.nf:

params.one = 'this is defined'

nextflow.config:

params.two = 'this is also defined'
$ nextflow config .

params {
   two = 'this is also defined'
}

This can be a little confusing sometimes, if the config is split between the two.

Phil

@drpatelh
Copy link
Contributor

drpatelh commented Feb 22, 2020

This came up again yesterday because nf-core lint is set to only parse nextflow.config for the standard parameters we require to be defined in the template.

e.g. I have actually defined the standard parameter in main.nf because it's mandatory and doesn't necessarily need to be initialised in nextflow.config. However, the linting still generates a warning.

@stale
Copy link

stale bot commented Apr 27, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Apr 27, 2020
@ewels
Copy link
Member Author

ewels commented Apr 27, 2020

@pditommaso - is this issue affected by #889?

@stale stale bot removed the wontfix label Apr 27, 2020
@pditommaso
Copy link
Member

Unfortunately no.

@stale
Copy link

stale bot commented Sep 24, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Sep 24, 2020
@stale stale bot closed this as completed Nov 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants