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

1 participant