Skip to content

Feature/upgrade nextflow#74

Merged
rernst merged 7 commits intodevelopfrom
feature/upgrade_nextflow
Aug 8, 2023
Merged

Feature/upgrade nextflow#74
rernst merged 7 commits intodevelopfrom
feature/upgrade_nextflow

Conversation

@rernst
Copy link
Member

@rernst rernst commented Jul 31, 2023

  • Support for nextflow v23.04.2
    • Changes are backward compatible, tested with (20.04.1.5335)

Copy link

@lvanoijen lvanoijen left a comment

Choose a reason for hiding this comment

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

Almost ok. There are some closing brackets missing. And looking at the files that aren't changed there is 1 val() change missed in the input of file GATK/4.1.3.0/SplitIntervals.nf.

@rernst rernst requested a review from lvanoijen August 8, 2023 14:39
@rernst
Copy link
Member Author

rernst commented Aug 8, 2023

Almost ok. There are some closing brackets missing. And looking at the files that aren't changed there is 1 val() change missed in the input of file GATK/4.1.3.0/SplitIntervals.nf.

Thanks, fixed the missing brackets and val() change. Also merged develop to fix merge conflict.

@rernst rernst merged commit 16b537a into develop Aug 8, 2023
@rernst rernst deleted the feature/upgrade_nextflow branch August 8, 2023 14:43
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