Merge pull request #16410 from rhatdan/codespell

Run codespell on code

Makefile

@@ -273,7 +273,7 @@ test/version/version: version/version.go
 
 .PHONY: codespell
 codespell:
-	codespell -S bin,vendor,.git,go.sum,.cirrus.yml,"RELEASE_NOTES.md,*.xz,*.gz,*.ps1,*.tar,swagger.yaml,*.tgz,bin2img,*ico,*.png,*.1,*.5,copyimg,*.orig,apidoc.go" -L clos,ans,pullrequest,uint,iff,od,seeked,splitted,marge,erro,hist,ether -w
+	codespell -S bin,vendor,.git,go.sum,.cirrus.yml,"RELEASE_NOTES.md,*.xz,*.gz,*.ps1,*.tar,swagger.yaml,*.tgz,bin2img,*ico,*.png,*.1,*.5,copyimg,*.orig,apidoc.go" -L te,clos,ans,pullrequest,uint,iff,od,seeked,splitted,marge,erro,hist,ether -w
 
 .PHONY: validate
 validate: lint .gitvalidation validate.completions man-page-check swagger-check tests-included tests-expect-exit pr-removes-fixed-skips

contrib/cirrus/CIModes.md

@@ -19,7 +19,7 @@ outdated) example of it's output can be seen below:
   While it's arguably easier to read that `only_if`, it leads to a cluttered
   status output that's harder to page through when reviewing PRs.  As opposed
   to `only_if` which will bypass creation of the task (at runtime) completely.
-  Also, by sticking to one conditional style, it's easer to re-use the YAML
+  Also, by sticking to one conditional style, it's easier to re-use the YAML
   statements across multiple tasks.
 
 + The only variables which can be used as part of conditions are defined by

contrib/podmanimage/README.md

@@ -1,6 +1,6 @@
 [comment]: <> (***ATTENTION*** ***WARNING*** ***ALERT*** ***CAUTION*** ***DANGER***)
 [comment]: <> ()
-[comment]: <> (ANY changes made to this file, once commited/merged must)
+[comment]: <> (ANY changes made to this file, once committed/merged must)
 [comment]: <> (be manually copy/pasted -in markdown- into the description)
 [comment]: <> (field on Quay at the following locations:)
 [comment]: <> ()

pkg/signal/signal_common_test.go

@@ -24,7 +24,7 @@ func TestParseSignal(t *testing.T) {
 			wantErr: false,
 		},
 		{
-			name: "Case doesnt matter",
+			name: "Case does not matter",
 			args: args{
 				rawSignal: "kIlL",
 			},

pkg/specgenutil/volumes_test.go

@@ -21,7 +21,7 @@ func Test_validChownFlag(t *testing.T) {
 			wantErr: false,
 		},
 		{
-			name: "U true case doesnt matter",
+			name: "U true case does not matter",
 			args: args{
 				flag: "u=True",
 			},

pkg/systemd/parser/unitfile.go

@@ -391,7 +391,7 @@ func (f *UnitFile) Parse(data string) error {
 		line, data = nextLine(data, 0)
 
 		// Handle multi-line continuations
-		// Note: This doesn't support coments in the middle of the continuation, which systemd does
+		// Note: This doesn't support comments in the middle of the continuation, which systemd does
 		if lineIsKeyValuePair(line) {
 			for len(data) > 0 && line[len(line)-1] == '\\' {
 				line, data = nextLine(origdata, len(line)+1)
@@ -437,7 +437,7 @@ func (g *unitGroup) write(w io.Writer) error {
 	}
 
 	if g.name == "" {
-		// Empty name groups are not valid, but used interally to handle comments in empty files
+		// Empty name groups are not valid, but used internally to handle comments in empty files
 		return nil
 	}
 
@@ -454,7 +454,7 @@ func (g *unitGroup) write(w io.Writer) error {
 	return nil
 }
 
-// Covert a UnitFile back to data, writing to the io.Writer w
+// Convert a UnitFile back to data, writing to the io.Writer w
 func (f *UnitFile) Write(w io.Writer) error {
 	for i, g := range f.groups {
 		// We always add a newline between groups, and strip one if it exists during
@@ -474,7 +474,7 @@ func (f *UnitFile) Write(w io.Writer) error {
 	return nil
 }
 
-// Covert a UnitFile back to data, as a string
+// Convert a UnitFile back to data, as a string
 func (f *UnitFile) ToString() (string, error) {
 	var str strings.Builder
 	if err := f.Write(&str); err != nil {