Skip to content

Commit

Permalink
fix spelling errors
Browse files Browse the repository at this point in the history
  • Loading branch information
lkingland committed Mar 6, 2024
1 parent a3a11f2 commit b72c961
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions cmd/list.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,9 +58,9 @@ Lists deployed functions.
// lister showing functions for all namespaces.
//
// This also extends to the treatment of the global setting for
// namespace. This is likwise intended for command which require a
// namespace. This is likewise intended for command which require a
// namespace no matter what. Therefore the global namespace setting is
// not applicable to this command beacause "deafult" really means "all".
// not applicable to this command because "default" really means "all".
//
// This is slightly different than other commands wherein their
// default is often to presume namespace "default" if none was either
Expand Down
6 changes: 3 additions & 3 deletions pkg/knative/deployer.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,8 @@ type Deployer struct {
decorator DeployDecorator
}

// ActiveNamespace attempts to read the kubernetes active namepsace.
// Missing configs or not having an active kuberentes configuration are
// ActiveNamespace attempts to read the Kubernetes active namespace.
// Missing configs or not having an active Kubernetes configuration are
// equivalent to having no default namespace (empty string).
func ActiveNamespace() string {
// Get client config, if it exists, and from that the namespace
Expand Down Expand Up @@ -155,7 +155,7 @@ func (d *Deployer) Deploy(ctx context.Context, f fn.Function) (fn.DeploymentResu
namespace = f.Deploy.Namespace
}
if namespace == "" {
return fn.DeploymentResult{}, fmt.Errorf("deployer requires either a target namespace or that the funciton be already deployed.")
return fn.DeploymentResult{}, fmt.Errorf("deployer requires either a target namespace or that the function be already deployed.")
}

// Clients
Expand Down

0 comments on commit b72c961

Please sign in to comment.