Maybe we should add a test to confirm clang compatibility? Some codes cannot be 
compiled with clang, but can be compiled with gcc.



------------------ ???????? ------------------
??????:                                                                         
                                               "dev"                            
                                                        <yiguo...@gmail.com&gt;;
????????:&nbsp;2022??4??25??(??????) ????4:13
??????:&nbsp;"dev"<dev@doris.apache.org&gt;;

????:&nbsp;[DISCUSS] Add requested CI check for Pull Requests



Hello everyone, with the increasing number of pull requests, we need to
strictly control the code merge process. I plan to turn on "requested" CI
checks, that is, when all "requested" CI checks of a PR pass, the code can
be merged. The following admissions will be forced on: 1. BE Code Formatter
/ Clang Formatter (pull_request): Code formatter for BE. 2. Build
Extensions / Build Extensions (pull_request): Compile for fs_broker/doc/. 3.
License Check / License Check (pull_request): License Header check. 4. BE
UT (Doris BE UT): Run BE unit test. 5. FE UT (Doris FE UT): Run FE unit
test. 6. P0 regression (Doris P0 regression): Compile FE/BE and run
regression test. PPMC members are obliged to manage the above CI
checks. Welcome
to discuss this, after unanimous agreement, I will apply to ASF Infra to
open requested CI.
-- 
????????????

??????
Tsing Hua University
Tel:134-3991-0228
Email:yiguo...@gmail.com

Reply via email to