You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by "torwig (via GitHub)" <gi...@apache.org> on 2023/03/19 20:42:23 UTC

[GitHub] [incubator-kvrocks] torwig opened a new pull request, #1334: Clang tidy warnings

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

   Fixed some clang's warnings.


-- 
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 commented on pull request #1334: Fix clang-tidy warnings

Posted by "PragmaTwice (via GitHub)" <gi...@apache.org>.
PragmaTwice commented on PR #1334:
URL: https://github.com/apache/incubator-kvrocks/pull/1334#issuecomment-1475556362

   Good job! Could you append `modernize-return-braced-init-list` to `WarningsAsErrors` in `.clang-tidy`?


-- 
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] torwig commented on pull request #1334: Fix clang-tidy warnings

Posted by "torwig (via GitHub)" <gi...@apache.org>.
torwig commented on PR #1334:
URL: https://github.com/apache/incubator-kvrocks/pull/1334#issuecomment-1475742564

   @PragmaTwice Update .clang-tidy and merged unstable in my branch.


-- 
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 #1334: Fix clang-tidy warnings

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


-- 
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