1395835 - Rename "group" to "category" in the scalar code and documentation

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement, P1)

Description

[Alessio Placitelli [:Dexter]]

Telemetry events are using the term "category" to refer to what in scalars is called "group name". We should have consistent naming and change all occurrences of "group"/"group name" in the scalar code and docs to "category"; it's a shorter and clearer term.

Here's the list of files to change:

* Scalars.yaml - http://searchfox.org/mozilla-central/source/toolkit/components/telemetry/Scalars.yaml
* scalars.rst - http://searchfox.org/mozilla-central/source/toolkit/components/telemetry/docs/collection/scalars.rst
* parse_scalars.py (and others) - http://searchfox.org/mozilla-central/source/toolkit/components/telemetry/parse_scalars.py
* TelemetryScalar.cpp/h - http://searchfox.org/mozilla-central/source/toolkit/components/telemetry/TelemetryScalar.cpp

Comment 1

[Alessio Placitelli [:Dexter]]

Attachment: Bug 1395835 - Rename "group" to "category" in the scalar code and docs.

Review commit: https://reviewboard.mozilla.org/r/183922/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/183922/

Comment 2

[Chris H-C :chutten]

Comment on attachment 8912597 [details]
Bug 1395835 - Rename "group" to "category" in the scalar code and docs.

https://reviewboard.mozilla.org/r/183922/#review189186

Comment 3

[Alessio Placitelli [:Dexter]]

Comment on attachment 8912597 [details]
Bug 1395835 - Rename "group" to "category" in the scalar code and docs.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/183922/diff/1-2/

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4413d40813bb
Rename "group" to "category" in the scalar code and docs. r=chutten

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/4413d40813bb