-
Notifications
You must be signed in to change notification settings - Fork 5
🤙Convention
yonghanJu edited this page Nov 9, 2022
·
3 revisions
💡
깃모지 커밋메세지
(optional)이슈유형: #이슈번호
ex)
✨ 로컬 캐싱기능 추가 Fixes: #45 Related to: #34, #23
커밋 종류 | 아이콘 | 코드 | 설명 | 원문 |
---|---|---|---|---|
Feat | ✨ | ✨ | 새 기능 | Introduce new features. |
Refactor | ♻️ | ♻️ | 코드 리팩토링 | Refactor code. |
Fix | 🐛 | 🐛 | 버그 수정 | Fix a bug. |
Rename | 🚚 | 🚚 | 리소스 이동, 이름 변경 | Move or rename resources (e.g: files paths routes). |
Remove | 🔥 | 🔥 | 코드/파일 삭제 | Remove code or files. |
Layout | 💄 | 💄 | UI/스타일 파일 추가/수정 | Add or update the UI and style files. |
Docs | 📝 | 📝 | 문서 추가/수정 | Add or update documentation. |
Test | ✅ | ✅ | 테스트 추가/수정 | Add or update tests. |
Poop | 💩 | 💩 | 똥 싼 코드 | Write bad code that needs to be improved. |
Chore | 🔧 | 🔧 | 위 사항들을 제외한 모든 경우 | In all cases except for the above. |
- 커밋 메시지는 한글로
- 커밋 전 변경 사항 꼭 확인
- 리베이스 ❌
Issue Convention
- 꼬리말은 optional이고 이슈 트래커 ID를 작성합니다.
- 꼬리말은 “유형: #이슈 번호” 형식으로 사용합니다.
- 여러 개의 이슈 번호를 적을 때는 쉼표로 구분합니다.
- 이슈 트래커 유형은 다음 중 하나를 사용합니다.
- Fixes: 이슈 수정중 (아직 해결되지 않은 경우)
- Resolves: 이슈를 해결했을 때 사용
- Ref: 참고할 이슈가 있을 때 사용
- Related to: 해당 커밋에 관련된 이슈번호 (아직 해결되지 않은 경우) ex)
Fixes: #45 Related to: #34, #23
💡 기본적인 코틀린 컨벤션은 안드로이드 공식문서 를 따름.
- 필요없는 주석 커밋 전 제거
-
!
대신.not()
사용 -
!!
사용 ❌ - Ktlint 플러그인 포맷팅
- main: 우리의 main 브랜치, upstream에 PR을 날릴 수 있는 branch
- feature/기능 이름: 작은 기능 하나를 개발하는 branch
- refactor/화면(기능): 기능 개발 이후 리팩토링을 수행하는 branch
💡 커밋 전 Code Formatting!! (
Ctrl + Alt + L
)
Warning 모두 해결하기
TestCode 작성
- Android Studio 버전 : Dolphin
- wrap column limit : 100 (기본값)
- minSDK : 23
- 런타임 시점에 권한 요청을 하기 위하여 23으로 선택
- targetSDK : 33
- 최신 버전이기 때문에 선택
- Single Import
- 불필요한 import를 피하기 위함
- Api key local properties로 관리
- key가 노출되는 것을 방지하기 위함