Fix spelling mistakes in comments, docs, and error strings (#3619)

* fix: correct spelling mistakes in comments, docs, and error strings

Fix ~104 spelling mistakes found by codespell across 65 files.
Fixes cover source comments, doc strings, markdown docs, error
messages, and test strings. Intentional misspellings in cobra
SuggestFor arrays and Kubernetes operator names are preserved.

Fixes #3618

* chore: regenerate embedded filesystem after template spelling fix

Author: Ankitsinghsisodya

cmd/build.go

@@ -304,7 +304,7 @@ func (c buildConfig) Configure(f fn.Function) fn.Function {
 func (c buildConfig) Prompt() (buildConfig, error) {
 	// If there is no registry nor explicit image name defined, the
 	// Registry prompt is shown whether or not we are in confirm mode.
-	// Otherwise, it is only showin if in confirm mode
+	// Otherwise, it is only shown if in confirm mode
 	// NOTE: the default in this latter situation will ignore the current function
 	// value and will always use the value from the config (flag or env variable).
 	// This is not strictly correct and will be fixed when Global Config: Function

cmd/client_test.go

@@ -9,7 +9,7 @@ import (
 
 // Test_NewTestClient ensures that the convenience method for
 // constructing a mocked client for testing properly considers options:
-// options provided to the factory constructor are considered exaustive,
+// options provided to the factory constructor are considered exhaustive,
 // such that the test can force the user of the factory to use specific mocks.
 // In other words, options provided when invoking the factory (such as by
 // a command implementation) are ignored.

cmd/create.go

@@ -187,7 +187,7 @@ func newCreateConfig(cmd *cobra.Command, args []string, newClient ClientFactory)
 	//
 	// If this were to provide more complexity or unreasonable maintainability
 	// in the future, we can move to ValidateFunctionName() function which will
-	// print effectively "Note: func names cant start with '-\..." on errors
+	// print effectively "Note: func names can't start with '-\..." on errors
 	if err := detectPrefixHyphen(cmd); err != nil {
 		return cfg, err
 	}
@@ -310,7 +310,7 @@ func (c createConfig) Validate(client *fn.Client) (err error) {
 	// Validate Runtime and Template Name
 	//
 	// Perhaps additional validation would be of use here in the CLI, but
-	// the client libray itself is ultimately responsible for validating all input
+	// the client library itself is ultimately responsible for validating all input
 	// prior to exeuting any requests.
 	// Client validates both language runtime and template exist, with language runtime
 	// being a mandatory flag while defaulting template if not present to 'http'.

cmd/delete_test.go

@@ -115,7 +115,7 @@ func TestDelete_ByName(t *testing.T) {
 	}
 }
 
-// TestDelete_Namespace ensures that remover is envoked when --namespace flag is
+// TestDelete_Namespace ensures that remover is invoked when --namespace flag is
 // given --> func delete myfunc --namespace myns
 func TestDelete_Namespace(t *testing.T) {
 	var (

cmd/deploy.go

@@ -452,7 +452,7 @@ func ValidateBuilder(name string) (err error) {
 // for use in flags and help text.
 func KnownBuilders() builders.Known {
 	// The set of builders supported by this CLI will likely always equate to
-	// the set of builders enumerated in the builders pacakage.
+	// the set of builders enumerated in the builders package.
 	// However, future third-party integrations may support less than, or more
 	// builders, and certain environmental considerations may alter this list.
 

cmd/deploy_test.go

@@ -399,7 +399,7 @@ func TestDeploy_Envs(t *testing.T) {
 		t.Fatalf("Expected envs '%v', got '%v'", expected, f.Run.Envs)
 	}
 
-	// Deploy the function with an additinal environment variable.
+	// Deploy the function with an additional environment variable.
 	cmd = NewDeployCmd(NewTestClient())
 	cmd.SetArgs([]string{"--env=ENV3=VAL3"})
 	if err = cmd.Execute(); err != nil {
@@ -851,7 +851,7 @@ func TestDeploy_ImageWithDigestErrors(t *testing.T) {
 // just be deployed as is (since it already has digest)
 func TestDeploy_ImageWithDigestDoesntPopulateBuild(t *testing.T) {
 	root := FromTempDirectory(t)
-	// image with digest (well almost, atleast in length and syntax)
+	// image with digest (well almost, at least in length and syntax)
 	const img = "example.com/username@sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855"
 	// Create a new Function in the temp directory
 	_, err := fn.New().Init(fn.Function{Runtime: "go", Root: root})
@@ -1101,7 +1101,7 @@ func TestDeploy_NamespaceDefaultsToK8sContext(t *testing.T) {
 		// NOTE: The below logic is expected of all deployers at this time,
 		// but is not necessary for this test.
 		// Deployer implementations should have integration tests which confirm
-		// this minimim namespace resolution logic is respected:
+		// this minimum namespace resolution logic is respected:
 		/*
 			if f.Namespace != "" {
 				// We deployed to the requested namespace
@@ -1580,7 +1580,7 @@ func testRegistryOrImageRequired(cmdFn commandConstructor, t *testing.T) {
 // are properly respected for all permutations, including empty.
 func TestDeploy_RemoteBuildURLPermutations(t *testing.T) {
 	// Valid flag permutations (empty indicates flag should be omitted)
-	// and a functon which will convert a permutation into flags for use
+	// and a function which will convert a permutation into flags for use
 	// by the subtests.
 	// The empty string indicates the case in which the flag is not provided.
 	var (
@@ -1957,7 +1957,7 @@ func TestDeploy_NoErrorOnOldFunctionNotFound(t *testing.T) {
 		// if it received an apiErrors.IsNotFound(err) and if so returns
 		// a fn.ErrFunctionNotFound.  This test implementation is dependent
 		// on that.  This is a change from the original implementation which
-		// directly returned a knative erorr with:
+		// directly returned a knative error with:
 		//   return apiErrors.NewNotFound(schema.GroupResource{Group: "", Resource: "Namespace"}, nsOne)
 		if ns == nsOne {
 			// Fabricate a not-found error.  For example if the function

cmd/func-util/main.go

@@ -208,7 +208,7 @@ func (d deployDecorator) UpdateLabels(function fn.Function, labels map[string]st
 
 func sh(ctx context.Context) error {
 	if !slices.Equal(os.Args[1:], []string{"-c", "umask 0000 && exec tar -xmf -"}) {
-		return fmt.Errorf("this is a fake sh (only for backward compatiblility purposes)")
+		return fmt.Errorf("this is a fake sh (only for backward compatibility purposes)")
 	}
 
 	wd, err := os.Getwd()

cmd/func-util/socat.go

@@ -135,7 +135,7 @@ func tryCloseWriteSide(c connection) {
 	if wc, ok := c.(writeCloser); ok {
 		err := wc.CloseWrite()
 		if err != nil {
-			fmt.Fprintf(os.Stderr, "waring: cannot close write side: %+v\n", err)
+			fmt.Fprintf(os.Stderr, "warning: cannot close write side: %+v\n", err)
 		}
 	}
 }

cmd/list_test.go

@@ -24,7 +24,7 @@ func TestList_Namespace(t *testing.T) {
 		err       bool   // expected error
 	}{
 		{
-			name:      "default (none specififed)",
+			name:      "default (none specified)",
 			namespace: "",
 			all:       false,
 			allShort:  false,

cmd/repository.go

@@ -344,7 +344,7 @@ func runRepositoryAdd(_ *cobra.Command, args []string, newClient ClientFactory)
 	// Create a client instance which utilizes the given repositories path.
 	// Note that this MAY not be in the config structure if the environment
 	// variable to override said path was provided explicitly.
-	// TODO: rectify this inconsitency:  the config default path structure will
+	// TODO: rectify this inconsistency:  the config default path structure will
 	// be created in XDG_CONFIG_HOME/func even if the repo path environment
 	// was set to some other location on disk.
 	client, done := newClient(ClientConfig{Verbose: cfg.Verbose})
@@ -514,7 +514,7 @@ func runRepositoryRemove(_ *cobra.Command, args []string, newClient ClientFactor
 		params.Name = args[0]
 	}
 	// "Are you sure" confirmation flag
-	// (not using name 'Confirm' to avoid confustion with cfg.Confirm)
+	// (not using name 'Confirm' to avoid confusion with cfg.Confirm)
 	// defaults to Yes.  This is debatable, but I don't want to choose the repo
 	// to remove and then have to see a prompt and then have to hit 'y'.  Just
 	// prompting once to make sure, which requires another press of enter, seems
@@ -587,7 +587,7 @@ func runRepositoryRemove(_ *cobra.Command, args []string, newClient ClientFactor
 // Installed repositories
 // All repositories which have been installed (does not include builtin)
 func installedRepositories(client *fn.Client) ([]string, error) {
-	// Client API contract stipulates the list always lists the defeault builtin
+	// Client API contract stipulates the list always lists the default builtin
 	// repo, and always lists it at index 0
 	repositories, err := client.Repositories().List()
 	if err != nil {
@@ -635,7 +635,7 @@ func newRepositoryConfig() (cfg repositoryConfig, err error) {
 	return
 }
 
-// prompt returns a config with values populated from interactivly prompting
+// prompt returns a config with values populated from interactively prompting
 // the user.
 func (c repositoryConfig) prompt() (repositoryConfig, error) {
 	// These prompts are overly verbose, as the user calling --confirm likely