Maybe we should add a test to confirm clang compatibility? Some codes cannot be
compiled with clang, but can be compiled with gcc.
------------------ ???????? ------------------
??????:
"dev"
<[email protected]>;
????????: 2022??4??25??(??????) ????4:13
??????: "dev"<[email protected]>;
????: [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:[email protected]