1676508 - JS categories should be snake_case not dotted.snake_case

Status: RESOLVED FIXED

(Toolkit :: Telemetry, task, P1)

Description

[Chris H-C :chutten]

If you go to about:telemetry, open the console, and look at the categories offered by the global Glean, you'll see:

GleanImpl { "test_only.ipc": GleanCategory, test_only: GleanCategory, fog: GleanCategory, "fog.ipc": GleanCategory }

Using dotted.snake_case categories isn't the most ergonomic:

Glean["fog.ipc"].replay_failures.add(1);

Instead, the categories should be transformed to snake_case:

Glean.fog_ipc.replay_failures.add(1);

This'll be a little trickier than just filtering through snake_case in the template like our Rust outputter does since JS also has the string tables to think about.

Be sure to update the pytests.

Comment 1

[Jan-Erik Rediger [:janerik]]

Attachment: Bug 1676508 - Use snake_case naming for categories in the JS API. r?chutten!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cd2a8ee2c844
Use snake_case naming for categories in the JS API. r=chutten

Comment 3

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/cd2a8ee2c844