lint

pkg/receive/handler.go

@@ -526,7 +526,7 @@ type peerGroup struct {
 	cache    map[string]storepb.WriteableStoreClient
 	m        sync.Mutex
 
-	// dialer is  used for testing
+	// dialer is used for testing.
 	dialer func(ctx context.Context, target string, opts ...grpc.DialOption) (conn *grpc.ClientConn, err error)
 }
 

pkg/receive/handler_test.go

@@ -143,15 +143,15 @@ func newHandlerHashring(appendables []*fakeAppendable, replicationFactor uint64)
 	}
 	var handlers []*Handler
 	// create a fake peer group where we manually fill the cache with fake addresses pointed to our handlers
-	// This removes the network from the tests and creates a more consistent testing harness
+	// This removes the network from the tests and creates a more consistent testing harness.
 	peers := &peerGroup{
 		dialOpts: nil,
 		m:        sync.Mutex{},
 		cache:    map[string]storepb.WriteableStoreClient{},
 		dialer: func(context.Context, string, ...grpc.DialOption) (*grpc.ClientConn, error) {
 			// dialer should never be called since we are creating fake clients with fake addresses
 			// this protects against some leaking test that may attempt to dial random IP addresses
-			// which may pose a security risk
+			// which may pose a security risk.
 			return nil, errors.New("unexpected dial called in testing")
 		},
 	}