[PATCH] D39603: [clang-tidy] Support relative paths in run-clang-tidy.py
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 6 00:59:53 PST 2017
hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.
The code looks good to me.
> I did not find any existing tests.
Yeah, we don't have good test strategy for these py files. We normally verify them by running them locally. I think we can land it as long as you test it locally (would be better to describe how you test it in the commit message).
================
Comment at: clang-tidy/tool/run-clang-tidy.py:232
+ files = [make_absolute(entry['file'], entry['directory'])
+ for entry in database]
----------------
nit: the code indention seems wrong here.
https://reviews.llvm.org/D39603
More information about the cfe-commits
mailing list