Add how to supply ext args with params

nf-core / website

Code and files for the main nf-core website.

https://nf-co.re

MIT License

69 stars 196 forks source link

Closed mahesh-panchal closed 6 days ago

mahesh-panchal commented 3 weeks ago

Add description to website how to use params to supply nextflow config in modules https://github.com/nf-core/modules/pull/6788

@netlify /docs/tutorials/tests_and_test_data/nf-test_writing_tests

netlify[bot] commented 3 weeks ago

Name	Link
Latest commit	65d8942ab5eab931d91d32700812d910b9589ac7
Latest deploy log	https://app.netlify.com/sites/nf-core-main-site/deploys/672e7d5b6011c40008e5370b
Deploy Preview	https://deploy-preview-2832--nf-core-main-site.netlify.app/docs/tutorials/tests_and_test_data/nf-test_writing_tests
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 3 weeks ago

Name	Link
Latest commit	65d8942ab5eab931d91d32700812d910b9589ac7
Latest deploy log	https://app.netlify.com/sites/nf-core-docs/deploys/672e7d5bd33a7800083a3fd0
Deploy Preview	https://deploy-preview-2832--nf-core-docs.netlify.app/docs/tutorials/tests_and_test_data/nf-test_writing_tests
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

jfy133 commented 3 weeks ago

And should add mention of this in teh modules specs

mashehu commented 1 week ago

More importantly do we have an issue to create linting around this?

what exactly should we lint for?

mahesh-panchal commented 1 week ago

More importantly do we have an issue to create linting around this?

what exactly should we lint for?

A single nextflow.config file in tests, and that it contains the string ext.args = params., and not ext.args = [{'"]

jfy133 commented 6 days ago

@mahesh-panchal you can merge in now, unless you want to wait for the linting test