1482912 - Replace cpp_guard with operating_systems for Events and Scalars

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement, P1)

Description

[Jan-Erik Rediger [:janerik]]

In bug 1477213 we switch the histogram parser from accepting `cpp_guards` to an array of `operating_systems` (as that was all it was used for).

We should switch over Events.yaml and Scalars.yaml to this as well.

Current status:
* Events.yaml: no use of cpp_guard
* Scalars.yaml: 2 uses, one of them not an OS (ACCESSIBILITY)

Comment 1

[Georg Fritzsche [:gfritzsche]]

This is probably not too hard, it would be great to get this difference in attributes cleared up not too late.
We might consider breaking this into two bugs and potentially mentoring them.

Comment 2

[Jan-Erik Rediger [:janerik]]

Attachment: Bug 1482912 - Replace cpp_guard with operating_systems for Events and Scalars

Comment 3

[Jan-Erik Rediger [:janerik]]

Attachment: Bug 1482912 - Fix warnings in doc generation

Drive-by fixes to annyoing warnings in mach doc for Telemetry docs.

Depends on D17231

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9e9085aee8bc
Replace cpp_guard with operating_systems for Events and Scalars r=chutten
https://hg.mozilla.org/integration/autoland/rev/d762822c538b
Fix warnings in doc generation r=gfritzsche,chutten

Comment 5

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/9e9085aee8bc
https://hg.mozilla.org/mozilla-central/rev/d762822c538b

Comment 6

[Mike de Boer [:mikedeboer]]

https://hg.mozilla.org/projects/cedar/rev/9e9085aee8bcf0fd6fc9b71cb30bf02eab563478
Bug 1482912 - Replace cpp_guard with operating_systems for Events and Scalars r=chutten

https://hg.mozilla.org/projects/cedar/rev/d762822c538bf374820e7c63c3b5658f297b1d78
Bug 1482912 - Fix warnings in doc generation r=gfritzsche,chutten