gaodayue opened a new issue #2790: Reformat all BE code according to clang-format and add code style check for PR URL: https://github.com/apache/incubator-doris/issues/2790 Now that Doris has .clang-format (thanks to #2724 ) to help code formatting and standardize C++ code style, let's reformat all BE code and integrate code style check into the CI. This has been requested before #1691 and now I think is the time to do it! I plan to accomplish the goal by the following steps 1. Refine `.clang-format` so that 1. the style rules are set based on the most popular choice in Doris so that we can minimize the diffs with existing code 2. the differences between Doris's style and Google's is made clear 2. Reformat code by modules and control each PR's size to make it reviewable. During the process, I'll check each file and use `// clang-format off` to disable formatting on certain files if the diff would otherwise be too large 3. Add automatically style check using github actions
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
