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