Updated comments to be consistent and correct

Author: aysylu

pkg/kritis/crd/authority/authority.go

@@ -28,8 +28,8 @@ import (
 type Lister func(namespace string) ([]v1beta1.AttestationAuthority, error)
 type Fetcher func(namespace string, name string) (v1beta1.AttestationAuthority, error)
 
-// Authorities returns all AttestationAuthority in the specified namespaces
-// Pass in an empty string to get all ISPs in all namespaces
+// Authorities returns all AttestationAuthorities in the specified namespaces
+// Pass in an empty string to get all AttestationAuthorities in all namespaces
 func Authorities(namespace string) ([]v1beta1.AttestationAuthority, error) {
 	config, err := rest.InClusterConfig()
 	if err != nil {
@@ -47,8 +47,8 @@ func Authorities(namespace string) ([]v1beta1.AttestationAuthority, error) {
 	return list.Items, nil
 }
 
-// Authority returns the AttestationAuthority in the specified namespace.
-// Returns error if AttestationAuthority not found
+// Authority returns the AttestationAuthority in the specified namespace and with the given name
+// Returns error if AttestationAuthority is not found
 func Authority(namespace string, name string) (*v1beta1.AttestationAuthority, error) {
 	config, err := rest.InClusterConfig()
 	if err != nil {

pkg/kritis/crd/buildpolicy/buildpolicy.go

@@ -29,8 +29,8 @@ import (
 // ValidateFunc defines the type for Validating Build Policies
 type ValidateFunc func(bp v1beta1.BuildPolicy, buildFrom string) error
 
-// BuildPolicies returns all ISP's in the specified namespaces
-// Pass in an empty string to get all ISPs in all namespaces
+// BuildPolicies returns all BuildPolicies in the specified namespaces
+// Pass in an empty string to get all BuildPolicies in all namespaces
 func BuildPolicies(namespace string) ([]v1beta1.BuildPolicy, error) {
 	config, err := rest.InClusterConfig()
 	if err != nil {
@@ -48,8 +48,8 @@ func BuildPolicies(namespace string) ([]v1beta1.BuildPolicy, error) {
 	return list.Items, nil
 }
 
-// ValidateBuildPolicy checks if an image satisfies BP requirements.
-// It returns an error if an image does not pass,
+// ValidateBuildPolicy checks if an image satisfies BuildPolicy requirements
+// It returns an error if an image does not pass
 func ValidateBuildPolicy(bp v1beta1.BuildPolicy, builtFrom string) error {
 	ok, err := regexp.MatchString(bp.Spec.BuildRequirements.BuiltFrom, builtFrom)
 	if err != nil {

pkg/kritis/crd/securitypolicy/securitypolicy.go

@@ -33,7 +33,7 @@ import (
 // ValidateFunc defines the type for Validating Image Security Policies
 type ValidateFunc func(isp v1beta1.ImageSecurityPolicy, image string, client metadata.Fetcher) ([]policy.Violation, error)
 
-// ImageSecurityPolicies returns all ISP's in the specified namespaces
+// ImageSecurityPolicies returns all ISPs in the specified namespaces
 // Pass in an empty string to get all ISPs in all namespaces
 func ImageSecurityPolicies(namespace string) ([]v1beta1.ImageSecurityPolicy, error) {
 	config, err := rest.InClusterConfig()