This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Add CLI flag "-clang-tidy" to enable/disable running clang-tidy checks.
ClosedPublic

Authored by hokein on Feb 5 2019, 4:40 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

hokein created this revision.Feb 5 2019, 4:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 5 2019, 4:40 AM
sammccall accepted this revision.Feb 5 2019, 11:28 PM
sammccall added inline comments.
clangd/tool/ClangdMain.cpp
214 ↗(On Diff #185282)

I thought we discussed making this init(false) for now, and flipping the flag after getting feedback?

(at head this is on, but not many people run from head)

This revision is now accepted and ready to land.Feb 5 2019, 11:28 PM
hokein updated this revision to Diff 185505.Feb 6 2019, 1:10 AM
hokein marked an inline comment as done.

Set the flag to false by default.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2019, 1:10 AM