fix: resolve comments

Signed-off-by: Junjie Gao <[email protected]>

revocation/crl/errors.go

@@ -15,5 +15,5 @@ package crl
 
 import "errors"
 
-// ErrCacheMiss is an error type for when a cache miss occurs
+// ErrCacheMiss is returned when a cache miss occurs.
 var ErrCacheMiss = errors.New("cache miss")

revocation/crl/fetcher.go

@@ -11,8 +11,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// Package crl provides Fetcher and Cache interface and implementations for
-// fetching CRLs.
+// Package crl provides Fetcher interface with its implementation, and the
+// Cache interface.
 package crl
 
 import (
@@ -100,7 +100,7 @@ func (f *HTTPFetcher) Fetch(ctx context.Context, url string) (*Bundle, error) {
 	return bundle, nil
 }
 
-// fetch downloads the CRL from the given URL and saves it to the cache
+// fetch downloads the CRL from the given URL.
 func (f *HTTPFetcher) fetch(ctx context.Context, url string) (*Bundle, error) {
 	// fetch base CRL
 	base, err := fetchCRL(ctx, url, f.httpClient)

revocation/internal/crl/crl.go

@@ -78,7 +78,7 @@ func CertCheckStatus(ctx context.Context, cert, issuer *x509.Certificate, opts C
 			Result: result.ResultUnknown,
 			ServerResults: []*result.ServerResult{{
 				RevocationMethod: result.RevocationMethodCRL,
-				Error:            errors.New("CRL fetcher is nil"),
+				Error:            errors.New("CRL fetcher cannot be nil"),
 				Result:           result.ResultUnknown,
 			}},
 			RevocationMethod: result.RevocationMethodCRL,

revocation/internal/crl/crl_test.go

@@ -48,8 +48,8 @@ func TestCertCheckStatus(t *testing.T) {
 			CRLDistributionPoints: []string{"http://example.com"},
 		}
 		r := CertCheckStatus(context.Background(), cert, &x509.Certificate{}, CertCheckStatusOptions{})
-		if r.ServerResults[0].Error.Error() != "CRL fetcher is nil" {
-			t.Fatalf("expected CRL fetcher is nil, got %v", r.ServerResults[0].Error)
+		if r.ServerResults[0].Error.Error() != "CRL fetcher cannot be nil" {
+			t.Fatalf("expected CRL fetcher cannot be nil, got %v", r.ServerResults[0].Error)
 		}
 	})