Skip to content

Commit

Permalink
More explicit errors regarding connecting to container URLs
Browse files Browse the repository at this point in the history
  • Loading branch information
adamrtalbot committed May 2, 2023
1 parent 7d87d6d commit e4cd14a
Showing 1 changed file with 16 additions and 6 deletions.
22 changes: 16 additions & 6 deletions nf_core/modules/lint/main_nf.py
Original file line number Diff line number Diff line change
Expand Up @@ -332,13 +332,23 @@ def check_process_section(self, lines, fix_version, progress_bar):
self.failed.append(("container_links", "Unable to connect to container URL", self.main_nf))
continue
if not response.ok:
self.warned.append(
(
"container_links",
f"Access to container {response.url} denied, status code: {response.status_code}",
self.main_nf,
if response.status_code in [400, 401, 402, 403]:
self.warned.append(
(
"container_links",
f"Access to container {response.url} denied, status code: {response.status_code}",
self.main_nf,
)
)

else:
self.failed.append(
(
"container_links",
f"Unable to connect to {response.url}, status code: {response.status_code}",
self.main_nf,
)
)
)

# Check that all bioconda packages have build numbers
# Also check for newer versions
Expand Down

0 comments on commit e4cd14a

Please sign in to comment.