Merge pull request #2945 from nf-core/handle-ssh-git-paths

Components: Handle extended set of git path prefixes

CHANGELOG.md

@@ -30,6 +30,8 @@
 
 ### Components
 
+- Handle more complete list of possible git URL forms (ssh:// and ftp:// prefixes specifically) ([#2945](https://github.com/nf-core/tools/pull/2945))
+
 ### General
 
 - Update CI to use nf-core/setup-nextflow v2

nf_core/modules/modules_utils.py

@@ -20,20 +20,19 @@ def repo_full_name_from_remote(remote_url: str) -> str:
     Extracts the path from the remote URL
     See https://mirrors.edge.kernel.org/pub/software/scm/git/docs/git-clone.html#URLS for the possible URL patterns
     """
-    # Check whether we have a https or ssh url
-    if remote_url.startswith("https"):
-        path = urlparse(remote_url).path
-        # Remove the intial '/'
-        path = path[1:]
-        # Remove extension
-        path = os.path.splitext(path)[0]
+
+    if remote_url.startswith(("https://", "http://", "ftps://", "ftp://", "ssh://")):
+        # Parse URL and remove the initial '/'
+        path = urlparse(remote_url).path.lstrip("/")
+    elif "git@" in remote_url:
+        # Extract the part after 'git@' and parse it
+        path = urlparse(remote_url.split("git@")[-1]).path
     else:
-        # Remove the initial `git@``
-        split_path: list = remote_url.split("@")
-        path = split_path[-1] if len(split_path) > 1 else split_path[0]
-        path = urlparse(path).path
-        # Remove extension
-        path = os.path.splitext(path)[0]
+        path = urlparse(remote_url).path
+
+    # Remove the file extension from the path
+    path, _ = os.path.splitext(path)
+
     return path