You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by "PragmaTwice (via GitHub)" <gi...@apache.org> on 2023/04/13 09:39:57 UTC

[GitHub] [incubator-kvrocks] PragmaTwice opened a new pull request, #1383: Enable readability-identifier-naming in clang-tidy

PragmaTwice opened a new pull request, #1383:
URL: https://github.com/apache/incubator-kvrocks/pull/1383

   We enable readability-identifier-naming in clang-tidy to check naming convension.
   
   Rules:
   ```
     - key:           readability-identifier-naming.LocalVariableCase
       value:         lower_case
     - key:           readability-identifier-naming.StructCase
       value:         CamelCase
     - key:           readability-identifier-naming.ClassCase
       value:         CamelCase
     - key:           readability-identifier-naming.UnionCase
       value:         CamelCase
     - key:           readability-identifier-naming.ParameterCase
       value:         lower_case
     # FIXME: enable these options
     # - key:           readability-identifier-naming.PublicMethodCase
     #   value:         CamelCase
     # - key:           readability-identifier-naming.PrivateMethodCase
     #   value:         camelBack
     # - key:           readability-identifier-naming.ProtectedMethodCase
     #   value:         camelBack
     - key:           readability-identifier-naming.LocalConstantCase
       value:         aNy_CasE  # FIXME: use a more strictly case
     - key:           readability-identifier-naming.ClassMemberCase
       value:         lower_case
     - key:           readability-identifier-naming.ClassMemberSuffix
       value:         _
     - key:           readability-identifier-naming.ClassConstantCase
       value:         aNy_CasE  # FIXME: use a more strictly case
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] PragmaTwice merged pull request #1383: Enable readability-identifier-naming in clang-tidy

Posted by "PragmaTwice (via GitHub)" <gi...@apache.org>.
PragmaTwice merged PR #1383:
URL: https://github.com/apache/incubator-kvrocks/pull/1383


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org