go sdk: Fix compile warnings (#7619)

Signed-off-by: Andrea Luzzardi <[email protected]>
dagger · Jun 11, 2024 · c7e3fd8 · c7e3fd8
1 parent e82f931
commit c7e3fd8
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/cmd/codegen/generator/go/templates/modules.go b/cmd/codegen/generator/go/templates/modules.go
@@ -355,7 +355,11 @@ func invokeSrc(objFunctionCases map[string][]Code, createMod Code) string {
 		Id("_").Id("any"),
 		// err error)
 		Id("err").Error(),
-	).Block(objSwitch)
+	).Block(
+		// suppress warning if `inputArgs` is unused
+		Id("_").Op("=").Id(inputArgsVar),
+		objSwitch,
+	)
 
 	return fmt.Sprintf("%#v", invokeFunc)
 }

diff --git a/sdk/go/querybuilder/querybuilder.go b/sdk/go/querybuilder/querybuilder.go
@@ -150,7 +150,9 @@ func (s *Selection) unpack(data interface{}) error {
 			if err != nil {
 				return err
 			}
-			json.Unmarshal(marshalled, s.bind)
+			if err := json.Unmarshal(marshalled, s.bind); err != nil {
+				return err
+			}
 		}
 	}
 

diff --git a/sdk/go/telemetry/exporters.go b/sdk/go/telemetry/exporters.go
@@ -89,7 +89,7 @@ func (m LogForwarder) Export(ctx context.Context, logs []sdklog.Record) error {
 		e := e
 		eg.Go(func() error {
 			for _, log := range logs {
-				e.OnEmit(ctx, log)
+				_ = e.OnEmit(ctx, log)
 			}
 			return nil
 		})