fix: forbidigo

.golangci.yaml

@@ -3,15 +3,14 @@ linters:
   disable:
     - exportloopref # Deprecated
     - execinquery # Deprecated
+    - exhaustruct # Very noisy, hard to disable when want zero values
     - gci
-    - forbidigo
     - gocritic
     - godox
     - gomnd
     - gochecknoglobals
     - gosec
     - ireturn
-    - exhaustruct
     - mnd
     - forcetypeassert
     - nlreturn

cmd/monitor/page_topics_messages.go

@@ -90,12 +90,12 @@ func (m *model_topic_messages) createSubscriber() {
 	subscriber, err := subscriber.New()
 	if err != nil {
 		subscriber.Delete()
-		fmt.Println("Failed to allocate new subscriber")
+		panic(fmt.Errorf("[Topic Messages]: %w", err))
 	}
 	err = subscriber.Create(m.mon.Topic_name, m.mon.Datatype)
 	if err != nil {
 		subscriber.Delete()
-		fmt.Println("Failed to create subscriber!")
+		panic(fmt.Errorf("[Topic Messages]: %w", err))
 	}
 	switch {
 	case m.mon.Datatype.Name == "std::string" && m.mon.Datatype.Encoding == "base":

cmd/monitor/tea.go

@@ -1,8 +1,7 @@
 package main
 
 import (
-	"fmt"
-	"os"
+	"log"
 	"strings"
 	"time"
 
@@ -130,7 +129,6 @@ func (m *model) View() string {
 func doCli() {
 	p := tea.NewProgram(newModel())
 	if _, err := p.Run(); err != nil {
-		fmt.Println("Error running program:", err)
-		os.Exit(1)
+		log.Fatal("Error running program:", err)
 	}
 }

internal/protobuf/protobuf.go

@@ -1,7 +1,7 @@
 package protobuf
 
 import (
-	"fmt"
+	"log"
 
 	"google.golang.org/protobuf/proto"
 	"google.golang.org/protobuf/reflect/protodesc"
@@ -65,7 +65,7 @@ func GetProtoMessageDescription(msg proto.Message) []byte {
 	getFileDescriptor(desc, &pset)
 	bytes, err := proto.Marshal(&pset)
 	if err != nil {
-		fmt.Println(err)
+		log.Println("WARN: GetProtoMessageDescription failed to marshal file descriptor set", err)
 		return nil
 	}
 	return bytes

main.go

@@ -15,8 +15,8 @@ import (
 
 func main() {
 	// eCAL version as string and semantic version components
-	fmt.Println(ecal.GetVersionString())
-	fmt.Println(ecal.GetVersion())
+	fmt.Println(ecal.GetVersionString()) //nolint:forbidigo
+	fmt.Println(ecal.GetVersion())       //nolint:forbidigo
 
 	// Initialize eCAL with default config, the unit name "Go eCAL",
 	// and the Publisher, Subscriber and Logging components enabled
@@ -102,13 +102,13 @@ func receiveMessages(s *subscriber.Subscriber) {
 	for {
 		msg, err := s.Receive(2 * time.Second)
 		if err == nil {
-			fmt.Println("Received:", msg)
+			fmt.Println("Received:", msg) //nolint:forbidigo
 		} else {
-			fmt.Println(err)
+			fmt.Println(err) //nolint:forbidigo
 		}
 	}
 }
 
 func registrationLogger(id registration.TopicId, event registration.Event) {
-	fmt.Println("Received registration sample:", id)
+	fmt.Println("Received registration sample:", id) //nolint:forbidigo
 }