From 16b09731424b57d0461092722bd5f157f96db2e4 Mon Sep 17 00:00:00 2001 From: Alex Zlotnik Date: Tue, 23 Oct 2018 11:32:16 -0700 Subject: [PATCH] Clarify the "ID" vs "id" argument --- CodeReviewComments.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CodeReviewComments.md b/CodeReviewComments.md index d72145af..d321b065 100644 --- a/CodeReviewComments.md +++ b/CodeReviewComments.md @@ -335,7 +335,7 @@ if err != nil { Words in names that are initialisms or acronyms (e.g. "URL" or "NATO") have a consistent case. For example, "URL" should appear as "URL" or "url" (as in "urlPony", or "URLPony"), never as "Url". As an example: ServeHTTP not ServeHttp. For identifiers with multiple initialized "words", use for example "xmlHTTPRequest" or "XMLHTTPRequest". -This rule also applies to "ID" when it is short for "identifier", so write "appID" instead of "appId". +This rule also applies to "ID" when it is short for "Identity Document" (which is pretty much all cases when it's not the "id" as in "ego", "superego"), so write "appID" instead of "appId". Code generated by the protocol buffer compiler is exempt from this rule. Human-written code is held to a higher standard than machine-written code.