nextflow-io / nextflow

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

Remove GA4GH plugin #5056

Closed pditommaso closed 3 months ago

pditommaso commented 3 months ago

This PR removes the GA4GH TES plugin from the Nextflow core codebase. The plugin source code will be maintained in the nf-ga4gh repository

netlify[bot] commented 3 months ago

Deploy Preview for nextflow-docs-staging ready!

Name Link
Latest commit cd1271398a36a9f35372bb99a7c664326c62fdee
Latest deploy log https://app.netlify.com/sites/nextflow-docs-staging/deploys/66673dcc3cbf2500082c1881
Deploy Preview https://deploy-preview-5056--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.

bentsherman commented 3 months ago

Why is the latest version 1.1.0-patch1 when the previous version was 1.3.0? Isn't that going to mess up the plugin version picker?

pditommaso commented 3 months ago

It should not. 1.3.0 is greater than 1.1.0-patch1

bentsherman commented 3 months ago

I'm confused. You made the standalone plugin 1.1.0 , so Nextflow will always select the "old" version 1.3.0 by default?

pditommaso commented 3 months ago

it should not, because it requires 23.10.2

https://github.com/nextflow-io/plugins/blob/main/plugins.json#L1214-L1218

bentsherman commented 3 months ago

so if I release a new version of the plugin, should I start from 1.1.0 or 1.4.0 ?

pditommaso commented 3 months ago

1.4.0