Attention is currently required from: msuraev.
2 comments:
File common/chapters/counters-overview.adoc:
The agreed upon limit is 120 which is way more convenient than 80.
I can see that it's more convenient for code, but IMHO text is more readable with lines that aren't as long. The coding standards says _up_to_ 120 characters, as I understand it's fine to use less characters in a line where it makes sense. I'd keep it at 80 characters for readability and so it's consistent with the rest of the file.
But this is cosmetics, if you insist then just ignore this review point.
Patch Set #2, Line 129: StatsD
The rest of the file can be fixed in a separate commit (unless we're talking about the name of the b […]
I mean, it would be good to have consistent spelling of it. in your patch its "statsd", "StatsD" and "statsD" ;)
So I suggest using the spelling that was already used in this document for consistency, which is "statsd". If you want to change it in the existing document, IMHO it should be done as separate patch before this one to keep consistency.
To view, visit change 30715. To unsubscribe, or for help writing mail filters, visit settings.