Closed Bug 1486729 Opened 4 years ago Closed 4 years ago

[Static-Analysis][Clang-Tidy] As default, a checker should be publish by default.

Categories

(Firefox Build System :: Source Code Analysis, enhancement)

enhancement
Not set
normal

Tracking

(firefox63 fixed)

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: andi, Assigned: andi)

Details

Attachments

(1 file)

As it can be seen from [1] most of the checkers have the attribute 'publish' set to 'yes' hence the default behavior should be to publish the checker and by this we eliminate all of the 'publish: !!bool yes'.

[1] https://dxr.mozilla.org/mozilla-central/source/tools/clang-tidy/config.yaml#15
Assignee: nobody → bpostelnicu
Attachment #9004522 - Attachment description: Bug 1486729 - [Static-Analysis][Clang-Tidy] As default a checker should be publish by default. r=janx → Bug 1486729 - [Static-Analysis][Clang-Tidy] As default, a checker should be publish by default. r=janx
Summary: [Static-Analysis][Clang-Tidy] As default a checker should be publish by default. → [Static-Analysis][Clang-Tidy] As default, a checker should be publish by default.
Comment on attachment 9004522 [details]
Bug 1486729 - [Static-Analysis][Clang-Tidy] As default, a checker should be publish by default. r=sylvestre

Sylvestre Ledru [:sylvestre] has approved the revision.
Attachment #9004522 - Flags: review+
Attachment #9004522 - Attachment description: Bug 1486729 - [Static-Analysis][Clang-Tidy] As default, a checker should be publish by default. r=janx → Bug 1486729 - [Static-Analysis][Clang-Tidy] As default, a checker should be publish by default. r=sylvestre
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/adbee46f307f
[Static-Analysis][Clang-Tidy] As default, a checker should be publish by default. r=sylvestre
https://hg.mozilla.org/mozilla-central/rev/adbee46f307f
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.