-
Notifications
You must be signed in to change notification settings - Fork 1
Conventions
SihunLee-1016 edited this page Jan 10, 2023
·
8 revisions
- 프로젝트에서 이슈 생성
- 이슈
develop
에서 분기하여 브런치 생성 - 코드 작업
-
develop
로 PR - code review
- Merge
위 작업 반복.
google c++ style guide를 따른다.
- reference와 포인터 모두 자료형에 붙혀서 작성.
⭕️ : int& foo;
❌ : int & bar; or int &bar;
- return 괄호 없이
- void 반환 함수 일 때, return 사용 필수!
- tab 간격은 4칸.
- doxygen (vscode extension) 을 사용하여 작성한다.
- 주석은 함수 내부 말고 함수 선언 위에 작성할 것.
- 클래스 생성 시 public, protected, private 순서로 작성
- define 말고 enum 사용하기
- 일반변수는 snake case
- 멤버변수는 snake case. 하지만 끝에 언더바 붙힘. (Ex. int this_is_example_)
- 모든 함수는 Pascal case. (Ex. void Example_Fun)
int num; // 일반변수 snake case
class example {
int test_; // 멤버변수 snake case + _
void Example_Function(); // 함수는 Pascal case
}
- commit은 아래 커밋 타입에 맞게 commit들을 분리한다.
- FEAT: 기능을 추가 또는 수정
- ENV: 개발 환경을 추가 또는 수정 (eslint 변경, dockerfile 변경 등)
- FIX: 버그를 해결
- DOCS: 문서를 수정 (README.md 변경, swagger)
- STYLE: 코드 스타일 변경 (prettier, npm run lint 등)
- REFACT: 코드를 리팩토링, 기능은 같고 코드가 변경
- TEST: 테스트 코드를 추가 또는 수정
- MERGE: 풀 리퀘스트 머지할 경우
- CHORE: 단순오타
- WIP: working in process 아직 작업중인 내용
ex)
FIX: 모델 validation 오류 수정
- Book title 제목 default 값 추가
- User intra 최소 길이 0으로 변경
ex)
FEAT: 로그인 기능 추가
- auth/ api 추가
ex)
TEST: bookController 테스트 코드 추가
- 책 제목에 대한 유효성 테스트 추가
- master(main): 배포된 가장 최신 브랜치입니다.
- develop: master에 issue브랜치가 머지되기 전에 거쳐가는 브랜치로, issue 브랜치에서 작업한 결과물이 develop 브랜치로 머지됩니다.
- {issue 번호}-{issue title} : 작업이 이뤄지는 이슈의 브런치 (깃허브에서 자동생성해주는 이름 사용)