This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Separate the check-facing interface
ClosedPublic

Authored by alexfh on Mar 22 2019, 1:26 PM.

Details

Summary

Move ClangTidyCheck to a separate header/.cpp
Switch checks to #include "ClangTidyCheck.h"
Mention ClangTidyCheck.h in the docs

Event Timeline

alexfh created this revision.Mar 22 2019, 1:26 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMar 22 2019, 1:26 PM
alexfh updated this revision to Diff 191937.Mar 22 2019, 1:30 PM
  • Reverted the <Switch checks to include "ClangTidyCheck.h"> part to simplify the review.
hokein accepted this revision.Mar 25 2019, 2:01 AM

LGTM

This revision is now accepted and ready to land.Mar 25 2019, 2:01 AM
This revision was automatically updated to reflect the committed changes.