do not exit migrate.py after successfully removing git user in Docker

This fix may have never worked. I don't understand as I know we tested extensively. Perhaps it updated and then migrate was run again by another process after the user was removed.
digitalmethodsinitiative · Feb 6, 2024 · aeb8090 · aeb8090
1 parent 3781e93
commit aeb8090
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/helper-scripts/migrate.py b/helper-scripts/migrate.py
@@ -254,10 +254,10 @@ def finish(args, logger):
 					logger.info("Error while fetching latest tags with git. Check that the repository URL is correct.")
 					logger.info(fetch.stderr)
 					exit(1)
-
-		logger.info("Error while fetching latest tags with git. Check that the repository URL is correct.")
-		logger.info(fetch.stderr)
-		exit(1)
+		else:
+			logger.info("Error while fetching latest tags with git. Check that the repository URL is correct.")
+			logger.info(fetch.stderr)
+			exit(1)
 
 	if args.branch:
 		logger.info(f"  ...checking out branch '{args.branch}'")