You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by "Bharath Vissapragada (Jira)" <ji...@apache.org> on 2020/06/26 17:00:00 UTC

[jira] [Created] (HBASE-24641) Add linter checks for patches

Bharath Vissapragada created HBASE-24641:
--------------------------------------------

             Summary: Add linter checks for patches
                 Key: HBASE-24641
                 URL: https://issues.apache.org/jira/browse/HBASE-24641
             Project: HBase
          Issue Type: Sub-task
          Components: Client, native-client
            Reporter: Bharath Vissapragada


I've worked with clang-tidy before and I think it works well. There is also a [helper script|https://clang.llvm.org/extra/doxygen/clang-tidy-diff_8py_source.html] that runs clang-tidy on a patch rather than the entire source tree.

- pull in clang as a dependency
- Define .clang-format
- Run clang-tidy-diff on each checkin

There are also other tools like cpplint from Google. I don't have any preference, so either works for me.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)