Skip to content

Commit 9562c44

Browse files
committed
chore: fix linting error for nolint comment
golangci/golangci-lint#3109 (comment)
1 parent 68c6657 commit 9562c44

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

internal/controller/csr_controller.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,8 @@ type CertificateSigningRequestReconciler struct {
7474
// Reconcile will perform a series of checks before deciding whether the CSR should be approved or denied
7575
// cyclomatic complexity is high (over 15), but this improves
7676
// readibility for the programmer, therefore we ignore the linting error
77-
// nolint: gocyclo
77+
//
78+
//nolint:gocyclo
7879
func (r *CertificateSigningRequestReconciler) Reconcile(ctx context.Context, req ctrl.Request) (res ctrl.Result, returnErr error) {
7980
l := log.FromContext(ctx)
8081

internal/controller/regex_ip_checks.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@ import (
1515
// DNSCheck is a function checking that the DNS name:
1616
// complies with the provider-specific regex
1717
// is resolvable (this check can be opted out with a parameter)
18-
// nolint: gocyclo
18+
//
19+
//nolint:gocyclo
1920
func (r *CertificateSigningRequestReconciler) DNSCheck(ctx context.Context, csr *certificatesv1.CertificateSigningRequest, x509cr *x509.CertificateRequest) (valid bool, reason string, err error) {
2021
if valid = (len(x509cr.DNSNames) <= r.AllowedDNSNames); !valid {
2122
reason = "The x509 Cert Request contains more DNS names than allowed through the config flag"

0 commit comments

Comments
 (0)