refactor: Removing useless errcheck for parse flag

Signed-off-by: Vincent Boutour <[email protected]>

cmd/fibr/config.go

@@ -51,7 +51,7 @@ type configuration struct {
 	disableAuth   bool
 }
 
-func newConfig() (configuration, error) {
+func newConfig() configuration {
 	fs := flag.NewFlagSet("fibr", flag.ExitOnError)
 	fs.Usage = flags.Usage(fs)
 
@@ -80,5 +80,7 @@ func newConfig() (configuration, error) {
 
 	flags.New("NoAuth", "Disable basic authentification").DocPrefix("auth").BoolVar(fs, &config.disableAuth, false, nil)
 
-	return config, fs.Parse(os.Args[1:])
+	_ = fs.Parse(os.Args[1:])
+
+	return config
 }

cmd/fibr/fibr.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"embed"
 	"fmt"
-	"log"
 	"log/slog"
 	"net/http"
 	"os"
@@ -39,11 +38,7 @@ func newLoginService(tracerProvider trace.TracerProvider, basicConfig *basicMemo
 }
 
 func main() {
-	config, err := newConfig()
-	if err != nil {
-		log.Fatal(fmt.Errorf("config: %s", err))
-	}
-
+	config := newConfig()
 	alcotest.DoAndExit(config.alcotest)
 
 	go func() {