Skip to content

Change get porcov version#285

Merged
replikation merged 2 commits intomasterfrom
change_get-porcov-version
Aug 4, 2025
Merged

Change get porcov version#285
replikation merged 2 commits intomasterfrom
change_get-porcov-version

Conversation

@MarieLataretu
Copy link
Collaborator

Hi there,
it's me again, again with the same line and I can't explain why and how.

With Nextflow 24.10.4, I get, sometimes, not always, a json/text returned that has spaces between the key-value pairs. This leads to an index out of bounds error, because the splitting does not work as expected.

Pulling replikation/poreCov ...
Launching `https://github.com/replikation/poreCov` [sleepy_fourier] DSL2 - revision: aa377dc669 [1.12.2]

________________________________________________________________________________
    
poreCov | A Nextflow SARS-CoV-2 workflow for nanopore data
    
ERROR ~ Index 1 out of bounds for length 1
 -- Check script '<myhome>/replikation/poreCov/poreCov.nf' at line: 50 or see '<foo>/poreCov.nextflow.log' file for more details

Excerpt from 'https://api.github.com/repos/replikation/poreCov/releases/latest'.toURL().text:

<...>
  "tag_name": "1.12.2", <--- ":<space>"
  "target_commitish": "master",
<...>

Anyhow, I changed the Groovy function to get the latest release.

I hope this solves the issue on my side, and works for anyone else, too.

@replikation
Copy link
Owner

uff okay. thanks. will test if its fine

@replikation replikation merged commit ab722c7 into master Aug 4, 2025
1 check passed
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