[PATCH] D89936: [clang-tidy] adding "--clang-tidy-config=<file-path>" to specify custom config file
Hiral via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 27 11:34:44 PDT 2020
Hiralo added a comment.
In D89936#2357088 <https://reviews.llvm.org/D89936#2357088>, @njames93 wrote:
> If you plan on contributing quite a lot then it would be wise to upload your patches with arcanist - https://llvm.org/docs/Phabricator.html. It will help prevent issues with diffs being relative to previous revisions.
> Personally I just create a branch from master for a feature. When I create the patch or update, I just need to use `arc diff master` and it will handle everything for me.
> The other advantage of using arc for patches is the pre-merge bot is then able to build and check your patch to make sure all tests pass.
Sure thanks for info.
will there be any update in this review after build error or successful merge ?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89936/new/
https://reviews.llvm.org/D89936
More information about the cfe-commits
mailing list