Skip to content

Commit

Permalink
Add apptainer keyword to the list of false positive matches. Bare min…
Browse files Browse the repository at this point in the history
…imum to possibly support a future apptainer.registry notation.
  • Loading branch information
MatthiasZepper committed Jun 30, 2023
1 parent cdf620e commit d2b8a45
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions nf_core/download.py
Original file line number Diff line number Diff line change
Expand Up @@ -733,8 +733,7 @@ def find_container_images(self, workflow_directory):
re.DOTALL is used to account for the string to be spread out across multiple lines.
"""
container_regex = re.compile(
r"container\s+[\s{}=$]*(?P<quote>[\'\"])(?P<param>(?:.(?!\1))*.?)\1[\s}]*",
re.DOTALL,
r"container\s+[\s{}=$]*(?P<quote>[\'\"])(?P<param>(?:.(?!\1))*.?)\1[\s}]*", re.DOTALL
)

local_module_findings = re.findall(container_regex, search_space)
Expand Down Expand Up @@ -850,7 +849,7 @@ def rectify_raw_container_matches(self, raw_findings):

for _, capture in container_value_defs:
# common false positive(s)
if capture in ["singularity"]:
if capture in ["singularity", "apptainer"]:
continue

# Look for a http download URL.
Expand Down

0 comments on commit d2b8a45

Please sign in to comment.