[PATCH] D39603: [clang-tidy] Support relative paths in run-clang-tidy.py

2017-11-06 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL317468: [clang-tidy] Support relative paths in run-clang-tidy.py (authored by xazax). Changed prior to commit: https://reviews.llvm.org/D39603?vs=121498&id=121705#toc Repository: rL LLVM https://rev

[PATCH] D39603: [clang-tidy] Support relative paths in run-clang-tidy.py

2017-11-06 Thread Haojian Wu via Phabricator via cfe-commits
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

[PATCH] D39603: [clang-tidy] Support relative paths in run-clang-tidy.py

2017-11-03 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun created this revision. Herald added subscribers: baloghadamsoftware, whisperity. Currently relative paths wasn't supperted by run-clang-tidy.py I added the support, however I did not find any existing tests. Is it ok for this to land without a test or should I introduce one? https://