Bulk spelling fixes #6242
Bulk spelling fixes #6242
Conversation
k8s-ci-robot
added
the
size/L
k8s-ci-robot
added
cncf-cla: yes
|
No rush on this one. Tip for anyone feeling keen though - switch to "Split" view in the github PR window. |
Experimenting with my own spelling checker, these are the typos it caught.
justinsb
force-pushed
the
bulk_fix_typos
branch
from
d6433ab to
26bd75a
Compare
|
Both /lgtm |
k8s-ci-robot
added
the
lgtm
| @@ -219,7 +219,7 @@ func (c *NodeupModelContext) BuildKubeConfig(username string, ca, certificate, p | |||
|
|
|||
| yaml, err := kops.ToRawYaml(config) | |||
| if err != nil { | |||
| return "", fmt.Errorf("error marshalling kubeconfig to yaml: %v", err) | |||
| return "", fmt.Errorf("error marshaling kubeconfig to yaml: %v", err) | |||
There was a problem hiding this comment.
Yeah, apparently one L is american english and two is british english. While I would love to use the british form as it looks more correct to my eyes, I think it's worth just having everything in one form for consistency, and it probably should be the american form. Even if that means no u in color or favorite :-(
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justinsb, mikesplain The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Experimenting with my own spelling checker, these are the typos it caught.