Repository navigation
Fix typos in README.md, main.go and pkg/hook/hook.go - #1010
kubernetes-prow[bot] merged 1 commit into
Conversation
README.md and main.go: "not specifid" -> "not specified" for the --exechook-timeout flag description. main.go: "right commmit" -> "right commit" in the worktree sync check. pkg/hook/hook.go: "closes said chanel" -> "closes said channel".
|
Welcome @Jah-yee! |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Jah-yee, sdowell The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What does this PR do?
Fixes three spelling mistakes in the git-sync repository:
README.md(line 297) andmain.go(line 2679) — the--exechook-timeoutflag description reads "If the timeout for the --exechook-command is not specifid"; changedspecifidtospecified.main.go(line 1531) — the comment above the worktree HEAD check reads "Make sure it is synced to the right commmit."; changedcommmittocommit.pkg/hook/hook.go(line 161) — the doc comment forsendOneTimeResultAndTerminatereads "closes said chanel"; changedchaneltochannel.Why
These are user-facing: the
--exechook-timeoutwording appears ingit-sync --helpoutput via themain.gousage text, sospecifidis visible to anyone reading the flag documentation.Testing
Documentation/comment-only change — no behavior is altered.
go build ./...andgo vet ./...pass locally on the branch.Notes for reviewers
The
README.mdflag table and themain.gousage string are kept in sync, so both were corrected together. Happy to split the commit or drop any part if you prefer a narrower change.