Closed
Bug 1492770
Opened 6 years ago
Closed 6 years ago
[Static-Analysis][Clang-Tidy] Activate google-* checkers that make sense to our code.
Categories
(Developer Infrastructure :: Source Code Analysis, enhancement)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox64 fixed)
RESOLVED
FIXED
mozilla64
| Tracking | Status | |
|---|---|---|
| firefox64 | --- | fixed |
People
(Reporter: andi, Assigned: andi)
Details
Attachments
(1 file)
Until now we haven't considered the google-*, we should evaluate these and add them to our configuration but in the first stage we shouldn't we shouldn't publish them. By not publishing them these will not be available in ci and but the static-analysis bot will use them but will not post results during review phase. In this way we can have a better understanding if these checkers help our cause.
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → bpostelnicu
|
|
Assignee | |
Comment 1•6 years ago
|
||
|
|
Assignee | |
Comment 2•6 years ago
|
||
Let's be sure we don't break the yaml file! https://treeherder.mozilla.org/#/jobs?repo=try&revision=253e9cae2a6cd2d408eec09c574dd7e5780bf0d7
|
||
Comment 3•6 years ago
|
||
Comment on attachment 9010632 [details] Bug 1492770 - Activate google-* checkers that make sense to our code. r=sylvestre Sylvestre Ledru [:sylvestre] has approved the revision.
Attachment #9010632 -
Flags: review+
Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/741465cf8a3c Activate google-* checkers that make sense to our code. r=sylvestre
|
||
Comment 5•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/741465cf8a3c
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
|
||
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•