다섯번 째 팀 회의

1. PR, issue 템플릿 생성

• 생성 완료!

2. 컨벤션

Git branch 전략

• github flow 전략
• 이슈 단위로 브랜치 생성 → 작업 → main으로 머지
• 대신 린팅 잘하기!
• main branch protect

브랜치 네이밍

• subject/#이슈번호

커밋 컨벤션

• 작업 효율을 위해 최대한 빠르게 완성할 수 있는 subject case 추천 ex FEAT → feat

• 커밋 단위는 되도록 작고 의미에 맞게 하기

Conventional Commits

A specification for adding human and machine readable meaning to commit messages

https://www.conventionalcommits.org/en/v1.0.0/

[Git] 커밋 메시지 규약 정리 (the AngularJS commit conventions)

이 문서는 the AngularJS commit conventions 를 번역한 것입니다. 🖋 번역 : outstandingboy 공부하면서 번역했습니다. 입맛대로 번역된 부분이나 오역이 있을 수 있습니다. Angular 9의 커밋 메시지 규약 을 추가했습니다. ✔ 스크립트로 CHANGELOG.md를 작성할 수 있다. ✔ git bisect를 사용하여 중요하지 않은 커밋을 무시하게 할 수 있다. (포매팅 같은 중요하지 않은 커밋) git bisect?

https://velog.io/@outstandingboy/Git-%EC%BB%A4%EB%B0%8B-%EB%A9%94%EC%8B%9C%EC%A7%80-%EA%B7%9C%EC%95%BD-%EC%A0%95%EB%A6%AC-the-AngularJS-commit-conventions

3. 코드 리뷰

Issue

• 이슈를 작업 단위로 최대한 잘게 나누자!

Pull Request

• issue 단위로 PR 생성

• File Changed 개수 ≤ 12

• PR 템플릿에 맞추어 작성

• 2명의 approve를 받아야 merge

리뷰

• 리뷰 룰
• PN 룰 : 코드 리뷰 in 뱅크샐러드 개발 문화
• 중요도를 구별할 수 있어서 사용하자

• 리뷰 팀
• 자신이 포함되지 않은 팀원 리뷰하기
• Sprint 1, 2 : 김재현, 김규란 / 우대현, 백민종
• Sprint 3 : 김재현, 우대현 / 김규란, 백민종
• Sprint 4 : 김규란, 우대현 / 김재현, 백민종

• 리뷰 짝은 리뷰 필수, 그 외는 최대한 모두 해주기 또는 크리티컬한 것만 찾아내주기

4. 직군 카테고리

• wanted 엑셀이 적당한 것 같다.

• 직군 - 직업
• select box 2개 만들기

• 더 고민해보자

5. 디렉토리 구조

• /api
• axios

• /app
• page

• /pages/api - API Routes

• /ui
• component
• /hooks

• /styles

• /utils

• /types

• /public
• favicon
• logo
• /images
• /icons