Skip to content

Fix typos in comments and option help text#481

Open
durvesh1992 wants to merge 1 commit into
facebook:mainfrom
durvesh1992:fix/comment-doc-typos
Open

Fix typos in comments and option help text#481
durvesh1992 wants to merge 1 commit into
facebook:mainfrom
durvesh1992:fix/comment-doc-typos

Conversation

@durvesh1992

Copy link
Copy Markdown

This batches a handful of high-confidence spelling fixes in code comments, a doc-comment, and one CLI option help string. All changes are comment / string-only — no logic, behavior, or public API changes. Verified by inspection.

File Before After
mcrouter/CarbonRouterInstance.h Class to to read CPU metrics Class to read CPU metrics
mcrouter/ConfigApi.h (x2) Name of this peice of config Name of this piece of config
mcrouter/lib/network/AsyncMcClientImpl.cpp successfull write successful write
mcrouter/lib/network/McServerSession.h must have occured must have occurred
mcrouter/lib/network/SocketConnector.cpp promise we will fullfill promise we will fulfill
mcrouter/mcrouter_options_list.h after an update occured after an update occurred
mcrouter/routes/StagingRoute.h metadata retreived from warm route metadata retrieved from warm route

British spellings (e.g. behaviour, cancelled) were intentionally left untouched.

Corrects a handful of high-confidence spelling errors in code
comments, a doc-comment, and one CLI option help string. These are
comment/string-only edits with no logic or behavior changes:

- CarbonRouterInstance.h: duplicated word "to to" -> "to"
- ConfigApi.h: "peice" -> "piece" (x2)
- AsyncMcClientImpl.cpp: "successfull" -> "successful"
- McServerSession.h: "occured" -> "occurred"
- SocketConnector.cpp: "fullfill" -> "fulfill"
- mcrouter_options_list.h: "occured" -> "occurred" (help text)
- StagingRoute.h: "retreived" -> "retrieved"
@meta-cla meta-cla Bot added the CLA Signed label Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant