gofmt 1.20 updates (style-only changes) (#110)

Signed-off-by: Jay Pipes <[email protected]>

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Author: jaypipes

apis/core/v1alpha1/identifiers.go

@@ -56,8 +56,9 @@ type ResourceFieldSelector struct {
 // type to provide more user friendly syntax for references using 'from' field
 // Ex:
 // APIIDRef:
-//   from:
-//     name: my-api
+//
+//	from:
+//	  name: my-api
 type AWSResourceReferenceWrapper struct {
 	From *AWSResourceReference `json:"from,omitempty"`
 }

apis/core/v1alpha1/secret.go

@@ -24,4 +24,4 @@ type SecretKeyReference struct {
 	k8scorev1.SecretReference `json:""`
 	// Key is the key within the secret
 	Key string `json:"key"`
-}
+}

pkg/compare/delta.go

@@ -43,16 +43,17 @@ func (d *Delta) DifferentAt(subject string) bool {
 //
 // For example, consider this code:
 //
-// if delta.DifferentAt("Spec.Tags") {
-//     if err = rm.SyncTags(ctx, desired, latest); err != nil {
-//         return nil, err
-//     }
-// }
-// if !delta.DifferentExcept("Spec.Tags") {
-//     // We don't want to proceed to call the ModifyDBInstance API since
-//     // no other resource fields have changed.
-//     return desired, nil
-// }
+//	if delta.DifferentAt("Spec.Tags") {
+//	    if err = rm.SyncTags(ctx, desired, latest); err != nil {
+//	        return nil, err
+//	    }
+//	}
+//
+//	if !delta.DifferentExcept("Spec.Tags") {
+//	    // We don't want to proceed to call the ModifyDBInstance API since
+//	    // no other resource fields have changed.
+//	    return desired, nil
+//	}
 //
 // might be placed in an sdk_update_pre_build_request custom code hook to
 // prevent the ModifyDBInstance call from being executed if the DBInstance's

pkg/compare/nil_test.go

@@ -43,4 +43,3 @@ func TestNilDifference(t *testing.T) {
 	require.True(compare.HasNilDifference(nullChan, nonNullChan))
 
 }
-

pkg/compare/path.go

@@ -51,14 +51,15 @@ func (p Path) Pop() {
 	}
 }
 
-//	Contains returns true if the supplied string, delimited on ".", matches
-//	p.parts up to the length of the supplied string.
-//		e.g. if the Path p represents "A.B":
-//			subject "A" -> true
-//			subject "A.B" -> true
-//			subject "A.B.C" -> false
-//			subject "B" -> false
-//			subject "A.C" -> false
+// Contains returns true if the supplied string, delimited on ".", matches
+// p.parts up to the length of the supplied string.
+//
+//	e.g. if the Path p represents "A.B":
+//		subject "A" -> true
+//		subject "A.B" -> true
+//		subject "A.B.C" -> false
+//		subject "B" -> false
+//		subject "A.C" -> false
 func (p Path) Contains(subject string) bool {
 	subjectSplit := strings.Split(subject, ".")
 

pkg/condition/condition_test.go

@@ -98,7 +98,7 @@ func TestConditionGetters(t *testing.T) {
 	assert.Equal(len(gotAll), 2)
 
 	conds = append(conds, &ackv1alpha1.Condition{
-		Type: ackv1alpha1.ConditionTypeReferencesResolved,
+		Type:   ackv1alpha1.ConditionTypeReferencesResolved,
 		Status: corev1.ConditionTrue,
 	})
 	r = &ackmocks.AWSResource{}

pkg/runtime/reconciler.go

@@ -1085,15 +1085,15 @@ func (r *resourceReconciler) getEndpointURL(
 // getResyncPeriod returns the period of the recurring reconciler process which ensures the desired
 // state of custom resources is maintained.
 // It attempts to retrieve the duration from the following sources, in this order:
-// 1. A resource-specific reconciliation resync period specified in the reconciliation resync
-//    configuration map (--reconcile-default-resync-seconds).
-// 2. A resource-specific requeue on success period specified by the resource manager factory.
-//    The resource manager factory is controller-specific, and thus this period is to specified
-//    by controller authors (using ack-generate).
-// 3. The default reconciliation resync period period specified in the controller binary flags.
-//    (--reconcile-resource-resync-seconds)
-// 4. The default resync period defined in the ACK runtime package. Defined in defaultResyncPeriod
-//    within the same file
+//  1. A resource-specific reconciliation resync period specified in the reconciliation resync
+//     configuration map (--reconcile-default-resync-seconds).
+//  2. A resource-specific requeue on success period specified by the resource manager factory.
+//     The resource manager factory is controller-specific, and thus this period is to specified
+//     by controller authors (using ack-generate).
+//  3. The default reconciliation resync period period specified in the controller binary flags.
+//     (--reconcile-resource-resync-seconds)
+//  4. The default resync period defined in the ACK runtime package. Defined in defaultResyncPeriod
+//     within the same file
 //
 // Each reconciler has a unique value to use. This function should only be called during the
 // instantiation of an AWSResourceReconciler and should not be called during the reconciliation