Fix lint

internal/certauthority/certauthority_test.go

@@ -36,7 +36,6 @@ var (
 )
 
 func TestLoad(t *testing.T) {
-
 	tests := []struct {
 		name    string
 		cert    string

internal/concierge/server/server.go

@@ -256,7 +256,7 @@ func getAggregatedAPIServerConfig(
 	return apiServerConfig, nil
 }
 
-// main returns an error instead of calling plog.Fatal to allow defer statements to run
+// main returns an error instead of calling plog.Fatal to allow defer statements to run.
 func main() error {
 	defer plog.Setup()()
 

internal/issuer/issuer.go

@@ -38,7 +38,7 @@ func (c ClientCertIssuers) Name() string {
 }
 
 func (c ClientCertIssuers) IssueClientCertPEM(username string, groups []string, ttl time.Duration) ([]byte, []byte, error) {
-	var errs []error
+	errs := make([]error, 0, len(c))
 
 	for _, issuer := range c {
 		certPEM, keyPEM, err := issuer.IssueClientCertPEM(username, groups, ttl)

internal/issuer/issuer_test.go

@@ -10,6 +10,7 @@ import (
 
 	"github.com/golang/mock/gomock"
 	"github.com/stretchr/testify/require"
+
 	"go.pinniped.dev/internal/mocks/issuermocks"
 )
 

internal/oidc/provider/federation_domain_issuer.go

@@ -70,17 +70,17 @@ func (p *FederationDomainIssuer) validate() error {
 	return nil
 }
 
-// Issuer returns the issuer
+// Issuer returns the issuer.
 func (p *FederationDomainIssuer) Issuer() string {
 	return p.issuer
 }
 
-// IssuerHost returns the issuerHost
+// IssuerHost returns the issuerHost.
 func (p *FederationDomainIssuer) IssuerHost() string {
 	return p.issuerHost
 }
 
-// IssuerPath returns the issuerPath
+// IssuerPath returns the issuerPath.
 func (p *FederationDomainIssuer) IssuerPath() string {
 	return p.issuerPath
 }