JackDrogon commented on PR #39937: URL: https://github.com/apache/arrow/pull/39937#issuecomment-1926312727
> Would you mind fix the clang-tidy here? > > ``` > /arrow/cpp/src/arrow/datum.cc:121: If an else has a brace on one side, it should have it on both [readability/braces] [5] > /arrow/cpp/src/arrow/datum.cc:138: If an else has a brace on one side, it should have it on both [readability/braces] [5] > Done processing /arrow/cpp/src/arrow/datum.cc > Total errors found: 2 > ``` It's confusing. I have formatted the code using clang-format-17. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
