lint

Author: tstromberg

pkg/datastore/client.go

@@ -55,8 +55,8 @@ type ClientOption func(*clientOptionsInternal)
 // clientOptionsInternal holds internal client configuration that can be modified by ClientOption.
 type clientOptionsInternal struct {
 	authConfig *auth.Config
-	baseURL    string
 	logger     *slog.Logger
+	baseURL    string
 }
 
 // WithEndpoint returns a ClientOption that sets the API base URL.

pkg/datastore/key_range_integration_test.go

@@ -2,6 +2,7 @@ package datastore
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"testing"
 )
@@ -43,7 +44,7 @@ func TestMock_KeyRangeQuery(t *testing.T) {
 	for {
 		key, err := it.Next(nil)
 		if err != nil {
-			if err == Done {
+			if errors.Is(err, Done) {
 				break
 			}
 			t.Fatalf("Next: %v", err)

pkg/datastore/test_helper.go

@@ -23,6 +23,7 @@ func TestOptions(metadataURL, apiURL string) []ClientOption {
 	}
 }
 
+// TestConfig creates client options for test configuration with the given URLs.
 // Deprecated: TestConfig is deprecated. Use TestOptions instead.
 func TestConfig(_ context.Context, metadataURL, apiURL string) []ClientOption {
 	return TestOptions(metadataURL, apiURL)