change the ground rule for code review.
[platform/core/uifw/libds.git] / README.md
1 # test-server
2 Requires:
3    c++17 standard feature support
4       - std::variant(>=gcc-7, >=clang4, >=MSVC19.10x)
5
6 # Ground Rule
7   ## Committer rule
8     - Committer는 Commit message와 Description을 영어로 작성한다.
9     - Committer는 Reviewer가 patch의 목적을 이해할 수 있도록 Commit message와 Description을 충실히 작성한다.
10     - Committer는 patch를 간단한 단일기능 혹은 단일 주제를 가지도록 만든다. 단일 주제를 가지는 Commit message를 작성할 수 있도록 한다.
11     - Committer는 코드 수정들을 가장 작은 단위의 patch들도 나누어 Commit한다.
12     - Committer는 2명 이상의 Reviewer들에게 리뷰 요청을 한다.
13     - Committer는 빌드 및 동작검증을 한 후에 PR을 요청한다.
14     - Committer는 PR에 하나 혹은 그 이상의 patch들을 포함할 수 있다.
15     - Committer는 patch를 SE의 C++ coding rule을 준수하여 작성한다. (C++ Coding Rule)
16   ## Reviewer rule
17     - Reviewer는 한글/영어로 review comment를 작성한다.
18     - Reviewer는 Merge가 될 수 있는 PR이면 Approve를 체크한다.
19     - Reviewer는 reviewer로 등록되면 working day 1일 이내에 리뷰하도록 노력한다.
20     - Reviewer는 Code에 대한 의견만 제시한다.
21     - Reviewer는 리뷰 시에는 반드시 Comment를 기록한다.
22   ## Merge rule
23     - Rebase and Submit를 원칙으로 한다.
24     - Approve가 두 개 이상일 경우에만 merge가 가능하다.
25