Skip to content

Commit

Permalink
Checks for any instance of quay.io in container name
Browse files Browse the repository at this point in the history
  • Loading branch information
adamrtalbot committed May 9, 2023
1 parent 27fb44a commit 46c1dea
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions nf_core/modules/lint/main_nf.py
Original file line number Diff line number Diff line change
Expand Up @@ -282,16 +282,16 @@ def check_process_section(self, lines, fix_version, progress_bar):
else:
self.failed.append(("docker_tag", "Unable to parse docker tag", self.main_nf))
docker_tag = NoneD
if l.startswith("quay.io/biocontainers") and not l.startswith("quay.io/biocontainers/mulled"):
if l.startswith("quay.io/"):
self.failed.append(
(
"container_links",
"'quay.io/biocontainers/' container prefix found, please use just 'biocontainers/' instead",
f"'quay.io/<org>/<container>:<tag>' container name found, please use just '<org>/<container>:<tag>' instead. Offending container name: {l}",
self.main_nf,
)
)
else:
self.passed.append(("container_links", "Container prefix is correct", self.main_nf))
self.passed.append(("container_links", f"Container prefix is correct", self.main_nf))

# Guess if container name is simple one (e.g. nfcore/ubuntu:20.04)
# If so, add quay.io as default container prefix
Expand Down

0 comments on commit 46c1dea

Please sign in to comment.