search

nextflow-io / nextflow

A DSL for data-driven computational pipelines
http://nextflow.io
Apache License 2.0
2.61k stars 605 forks source link

Move Wave docs to Wave documentation #5072

Open jason-seqera opened 1 week ago

jason-seqera commented 1 week ago

This PR shifts the Wave-specific guides over to the Wave repo and the Wave documentation site at: https://docs.seqera.io/wave

netlify[bot] commented 1 week ago

Deploy Preview for nextflow-docs-staging ready!

Name Link
Latest commit 07f182e3e3b1d827eba5933135d38bae3ff4d003
Latest deploy log https://app.netlify.com/sites/nextflow-docs-staging/deploys/66709ba7b13b2e0008b9194e
Deploy Preview https://deploy-preview-5072--nextflow-docs-staging.netlify.app
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.

pditommaso commented 1 week ago

I think we should keep on hold until we don't have the new wave docs ready

ewels commented 1 week ago

Ah I assumed that there was a paired PR to add these back into the Seqera docs. Yes agree, they should be added there before they're removed here.

jason-seqera commented 1 week ago

@pditommaso @ewels - We could move over these tutorials in the meantime, though, and then that portion of the project would be complete. None of these specifically depend on the overall content refresh being complete.

ewels commented 1 week ago

Aha - it's here! https://github.com/seqeralabs/wave/pull/516

Sorry, didn't see it before. Yes if we can merge that first ☝🏻 then the docs will appear at docs.seqera.io. Then I think it's fine to merge this PR as the content will still be available and we can point there.

So just a question of sequencing the merges correctly from my point of view.

@pditommaso is it not better to do this before before the big Wave docs rewrite? I feel like this content should probably be included in the docs review for the new wave docs, rather than being tacked on afterwards...