From 77d17d97b56cfa3919672efab6613a8e51d91f64 Mon Sep 17 00:00:00 2001
From: Simon Cornet <simon@simoncor.net>
Date: Tue, 4 Mar 2025 06:37:43 +0100
Subject: [PATCH] feat: comply with linter for the first time

---
 cmd/gogitlabber/git.go    |  5 +++--
 cmd/gogitlabber/output.go | 11 +++++++++--
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/cmd/gogitlabber/git.go b/cmd/gogitlabber/git.go
index 60fec43..b57efc3 100644
--- a/cmd/gogitlabber/git.go
+++ b/cmd/gogitlabber/git.go
@@ -49,18 +49,19 @@ func checkoutRepositories(repositories []Repository) {
 				log.Printf("error: %v", err)
 			}
 			clonedCount = clonedCount + 1
-			bar.Add(1)
+			progressBarAdd(1)
 
 			// pull the latest
 		} else if strings.Contains(string(repoStatus), url) {
 			pullRepository(repoName, repoDestination)
-			bar.Add(1)
+			progressBarAdd(1)
 
 			// report error if not cloned or pulled repository
 		} else {
 			log.Printf("error: decided not to clone or pull repository %v\n", repoName)
 			log.Printf("error: this is why: %v\n", repoStatus)
 			errorCount = errorCount + 1
+			progressBarAdd(1)
 		}
 	}
 }
diff --git a/cmd/gogitlabber/output.go b/cmd/gogitlabber/output.go
index 12742ea..f199393 100644
--- a/cmd/gogitlabber/output.go
+++ b/cmd/gogitlabber/output.go
@@ -2,6 +2,7 @@ package main
 
 import (
 	"fmt"
+	"log"
 
 	"github.com/k0kubun/go-ansi"
 	"github.com/schollz/progressbar/v3"
@@ -13,9 +14,9 @@ func progressBar(repositories []Repository) {
 	repoCount := len(repositories)
 
 	// make progressbar
-	barPrefix := fmt.Sprintf("Getting your one and only repository...")
+	barPrefix := "Getting your one and only repository..."
 	if repoCount > 1 {
-		barPrefix = fmt.Sprintf("Getting your repositories...")
+		barPrefix = "Getting your repositories..."
 	}
 
 	bar = progressbar.NewOptions(
@@ -38,6 +39,12 @@ func progressBar(repositories []Repository) {
 	)
 }
 
+func progressBarAdd(amount int) {
+	if err := bar.Add(amount); err != nil {
+		log.Printf("Progress bar update error: %v", err)
+	}
+}
+
 func printSummary() {
 
 	fmt.Println("")