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 documentation for using cross-organisational components #2816

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

jvfe
Copy link
Member

@jvfe jvfe commented Oct 24, 2024

This documentation refers to the feature to be implemented through PR nf-core/tools#3083, which enables subworkflows with components from more than one remote (i.e. organisations). More about the feature itself and implementation details can be found there.

The current structure for the documentation was based on a suggestion in the original feature PR.

I'll leave this PR as a draft until the feature gets approved for merging.

CC @muffato who has been assisting me with both the feature and the documentation for it.

@netlify /docs/tutorials/external_usage/cross_organisational_subworkflows

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for nf-core-docs ready!

Name Link
🔨 Latest commit 8128927
🔍 Latest deploy log https://app.netlify.com/sites/nf-core-docs/deploys/671b6cfff8d493000827b864
😎 Deploy Preview https://deploy-preview-2816--nf-core-docs.netlify.app/docs/tutorials/external_usage/cross_organisational_subworkflows
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link

netlify bot commented Oct 24, 2024

Deploy Preview for nf-core-main-site ready!

Name Link
🔨 Latest commit 8128927
🔍 Latest deploy log https://app.netlify.com/sites/nf-core-main-site/deploys/671b6cff54984c00080e833d
😎 Deploy Preview https://deploy-preview-2816--nf-core-main-site.netlify.app/docs/tutorials/external_usage/cross_organisational_subworkflows
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Comment on lines +38 to +50
If you're using cross-organisational subworkflows in your repository, be aware that you must also specify a different JSON schema
to lint the `meta.yml` files for your subworkflows.
To do this, just change the `components` section type to allow both objects and strings.

```json
"components": {
"type": "array",
"description": "Modules and subworkflows used in the subworkflow",
"items": {
"type": ["object", "string"]
},
"minItems": 0
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's easier to just add these changes to the schema in the modules repo

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It won`t be added to the main modules repo, I believe, but I am adding it to the nf-core-test repo. I can remove this section and just link to the file there.

Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
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.

2 participants