chore: Fix spelling errors (#444)

Spell checking done using codespell.

Correct spelling is a great way to look more professional and demonstrate higher quality. And it catches a surprisingly number of actual errors, too.

Following up on aws-controllers-k8s/iam-controller#78

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

Author: candrews

.codespellrc

@@ -0,0 +1,6 @@
+[codespell]
+skip = .codespellrc,./.git,./go.local.sum,./pkg/testdata
+ignore-words-list = specfield,uptodate,fpr
+check-filenames =
+check-hidden =
+quiet = 2

ATTRIBUTION.md

@@ -95,7 +95,7 @@ syntax/tree.go (from RegexTree.cs and RegexNode.cs): ported literally as possibl
 
 syntax/writer.go (from RegexWriter.cs): ported literally with minor changes to make it more Go-ish.
 
-match.go (from RegexMatch.cs): ported, simplified, and changed to handle Go's lack of inheritence.
+match.go (from RegexMatch.cs): ported, simplified, and changed to handle Go's lack of inheritance.
 
 regexp.go (from Regex.cs and RegexOptions.cs): conceptually serves the same "starting point", but is simplified 
     and changed to handle differences in C# strings and Go strings/runes.  

pkg/config/resource.go

@@ -130,7 +130,7 @@ type SyncedConfig struct {
 	When []SyncedCondition `json:"when"`
 }
 
-// SyncedCondition represent one of the unique condition that should be fullfiled in
+// SyncedCondition represent one of the unique condition that should be fulfilled in
 // order to assert whether a resource is synced.
 type SyncedCondition struct {
 	// Path of the field. e.g Status.Processing
@@ -343,7 +343,7 @@ type AdditionalColumnConfig struct {
 }
 
 // PrintConfig informs instruct the code generator on how to sort kubebuilder
-// printcolumn marker coments.
+// printcolumn marker comments.
 type PrintConfig struct {
 	// AddAgeColumn a boolean informing the code generator whether to append a kubebuilder
 	// marker comment to show a resource Age (created since date) in `kubectl get` response.

pkg/generate/code/compare.go

@@ -250,7 +250,7 @@ func CompareResource(
 
 // compareNil outputs Go code that compares two field values for nullability
 // and, if there is a nil difference, adds the difference to a variable
-// represeting the `ackcompare.Delta`
+// representing the `ackcompare.Delta`
 //
 // Output code will look something like this:
 //

pkg/metadata/generation_metadata.go

@@ -60,7 +60,7 @@ type GenerationMetadata struct {
 	LastModification lastModificationInfo `json:"last_modification"`
 	// AWS SDK Go version used generate the APIs
 	AWSSDKGoVersion string `json:"aws_sdk_go_version"`
-	// Informatiom about the ack-generate binary used to generate the APIs
+	// Information about the ack-generate binary used to generate the APIs
 	ACKGenerateInfo ackGenerateInfo `json:"ack_generate_info"`
 	// Information about the generator config file used to generate the APIs
 	GeneratorConfigInfo generatorConfigInfo `json:"generator_config_info"`

pkg/model/model_apigwv2_test.go

@@ -75,7 +75,7 @@ func TestAPIGatewayV2_Api(t *testing.T) {
 	assert.NotNil(crd.SpecFields["Basepath"])
 	assert.NotNil(crd.SpecFields["FailOnWarnings"])
 
-	// The required property should get overriden for Name and ProtocolType fields.
+	// The required property should get overridden for Name and ProtocolType fields.
 	assert.False(crd.SpecFields["Name"].IsRequired())
 	assert.False(crd.SpecFields["ProtocolType"].IsRequired())
 

pkg/model/multiversion/delta.go

@@ -110,7 +110,7 @@ func ComputeFieldDeltas(
 ) ([]FieldDelta, error) {
 	deltas := []FieldDelta{}
 
-	// collect field names and sort them to ensure a determenistic output order.
+	// collect field names and sort them to ensure a deterministic output order.
 	srcNames := []string{}
 	for name := range srcFields {
 		srcNames = append(srcNames, name)
@@ -134,7 +134,7 @@ func ComputeFieldDeltas(
 		dstField, ok := dstFields[srcName]
 		// If a field is found in both arrays only three changes are possible:
 		// None, TypeChange and ChangeTypeShapeChangedToSecret.
-		// NOTE(a-hilaly): carefull about X -> Y then Z -> X renames. It should
+		// NOTE(a-hilaly): careful about X -> Y then Z -> X renames. It should
 		// not be allowed.
 		if ok {
 			// mark field as visited.
@@ -188,7 +188,7 @@ func ComputeFieldDeltas(
 			dstField, ok2 := dstFields[newName]
 			if !ok2 {
 				// if a field was renamed and we can't find it in dstNames, something
-				// very wrong happend during CRD loading.
+				// very wrong happened during CRD loading.
 				return nil, fmt.Errorf("cannot find renamed field %s " + newName)
 			}
 

pkg/model/multiversion/delta_renames_test.go

@@ -137,7 +137,7 @@ func Test_computeRenames(t *testing.T) {
 				return
 			}
 
-			// Since 1.12 formating functions prints maps in key-sorted order.
+			// Since 1.12 formatting functions prints maps in key-sorted order.
 			// See https://golang.org/doc/go1.12#fmt
 			if fmt.Sprintf("%v", got) != fmt.Sprintf("%v", tt.want) {
 				t.Errorf("computeRenamesDelta() = %v, want %v", got, tt.want)

pkg/model/multiversion/manager.go

@@ -34,7 +34,7 @@ var (
 )
 
 // APIVersionManager is a API versions manager. It contains the mapping
-// of each non-deprecated version with their correspending ackmodel.Model
+// of each non-deprecated version with their corresponding ackmodel.Model
 // and APIInfos.
 type APIVersionManager struct {
 	gitRepo  *git.Repository

pkg/model/op_test.go

@@ -121,8 +121,8 @@ func TestGetOpTypeAndResourceNameFromOpID(t *testing.T) {
 		},
 	}
 	for _, test := range tests {
-		ot, resName := model.GetOpTypeAndResourceNameFromOpID(test.opID, g.GetConfig())
-		assert.Equal(test.expOpType, ot, test.opID)
+		opType, resName := model.GetOpTypeAndResourceNameFromOpID(test.opID, g.GetConfig())
+		assert.Equal(test.expOpType, opType, test.opID)
 		assert.Equal(test.expResName, resName, test.opID)
 	}
 }
@@ -154,8 +154,8 @@ func TestGetOpTypeAndResourceNameFromOpID_PluralSingular(t *testing.T) {
 		},
 	}
 	for _, test := range tests {
-		ot, resName := model.GetOpTypeAndResourceNameFromOpID(test.opID, g.GetConfig())
-		assert.Equal(test.expOpType, ot, test.opID)
+		opType, resName := model.GetOpTypeAndResourceNameFromOpID(test.opID, g.GetConfig())
+		assert.Equal(test.expOpType, opType, test.opID)
 		assert.Equal(test.expResName, resName, test.opID)
 	}
 }