fix: self.sha must not be none to reset to none

sanger-tol · Oct 28, 2024 · 65aa1c4 · 65aa1c4
1 parent 959a63c
commit 65aa1c4
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/nf_core/components/update.py b/nf_core/components/update.py
@@ -104,14 +104,14 @@ def update(self, component=None, silent=False, updated=None, check_diff_exist=Tr
             self.modules_repo = ModulesRepo(remote_url, branch)
             component = component["name"]
 
-        if self.current_remote is None:
-            self.current_remote = self.modules_repo.remote_url
-
-        if self.original_remote != self.modules_repo.remote_url and self.sha is None:
+        if self.original_remote != self.modules_repo.remote_url and self.sha is not None:
             self.current_sha = None
         else:
             self.current_sha = self.sha
 
+        if self.current_remote is None:
+            self.current_remote = self.modules_repo.remote_url
+
         self.component = component
         if updated is None:
             updated = []