Skip to content

Commit 32a7e6b

Browse files
Apply suggestions from code review
Co-authored-by: Matthias Hörtenhuber <[email protected]>
1 parent 3d953fe commit 32a7e6b

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

nf_core/pipelines/download.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -995,15 +995,15 @@ def prioritize_direct_download(self, container_list: List[str]) -> List[str]:
995995
A regex that matches http, r"^$|^http" could thus be used to prioritize the Docker URIs over http Downloads
996996
997997
We also need to handle a special case: The https:// Singularity downloads from Seqera Containers all end in 'data', although
998-
they are not equivalent:
998+
they are not equivalent, e.g.:
999999
10001000
'https://community-cr-prod.seqera.io/docker/registry/v2/blobs/sha256/63/6397750e9730a3fbcc5b4c43f14bd141c64c723fd7dad80e47921a68a7c3cd21/data'
10011001
'https://community-cr-prod.seqera.io/docker/registry/v2/blobs/sha256/c2/c262fc09eca59edb5a724080eeceb00fb06396f510aefb229c2d2c6897e63975/data'
10021002
10031003
"""
1004-
d: dict[str, str] = {}
1005-
seqera_containers: list[str] = []
1006-
all_others: list[str] = []
1004+
d: Dict[str, str] = {}
1005+
seqera_containers: List[str] = []
1006+
all_others: List[str] = []
10071007

10081008
for c in container_list:
10091009
if bool(re.search(r"/data$", c)):

0 commit comments

Comments
 (0)