Closed Bug 1364362 Opened 4 years ago Closed 4 years ago

Add doc link to error message when "record_in_processes" is missing

Categories

(Toolkit :: Telemetry, enhancement, P1)

enhancement
Points:
1

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: gfritzsche, Assigned: gfritzsche)

References

Details

Attachments

(1 file, 2 obsolete files)

Currently this prints:
> Error processing histograms:
> Histogram "A11Y_INSTANTIATED_FLAG" must have a "record_in_processes" field.

As this will be new for most developers, we should change this to something like:
> Error processing histograms:
> Histogram "A11Y_INSTANTIATED_FLAG" must have a "record_in_processes" field.
> See: http://gecko.readthedocs.io/en/latest/toolkit/components/telemetry/telemetry/collection/histograms.html#record-in-processes
Attachment #8867090 - Attachment is obsolete: true
Attachment #8867090 - Flags: review?(chutten)
Attachment #8867124 - Flags: review?(chutten) → review+
Attachment #8867124 - Attachment is obsolete: true
Attachment #8867217 - Flags: review+
Keywords: checkin-needed
Needs rebasing.
Keywords: checkin-needed
nvm, just needed to land on top of bug 1364393
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c60f2a5a69af
Add doc link for Histograms.json record_in_processes errors. r=chutten
https://hg.mozilla.org/mozilla-central/rev/c60f2a5a69af
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.