-
Notifications
You must be signed in to change notification settings - Fork 2
Team Rule
- ์ง์ฑ์ธ์ผ๋ก์ ํ๋ํ๋ค
- ๊ฒฝ์ด๋ฅผ ์ฌ์ฉํ๋ค.
- ๋ถ์ ์ ์ธ ์ธ์ด๋ฅผ ์ฌ์ฉํ์ง ์๋๋ค
- ๊ธ์ ์ ์ธ ๋ง์ธ๋
- 9:00 am ์งํ์ํฉ ๋ณด๊ณ ๋ฐ ๊ธ์ผ ๋ชฉํ ์ค์ & ๋ ํฌ ์ฑํฌ ๋ง์ถ๊ธฐ
- 12:00 pm ~ 13:00 pm ์ ์ฌ ์๊ฐ
- 13:00 pm ~ 14:00 pm ์ค๊ฐ ๋ฏธํ
- 14:00 pm ~ 18:00 pm Dev-log ์์ฑ,์ฝ๋ ๋ฆฌ๋ทฐ ๋ฐ KPT ์์ฑ
- 18:00 pm ~ 20:00 pm ์ ๋ ์๊ฐ ๋ฐ ๊ฐ๋จํ ํด์ ๋ฐ ์ด๋
-
๋ฆ์ด๋ 1์์ ์ทจ์นจํ๋ค
-
์ค๊ฐ์ค๊ฐ ํํํ ์คํธ๋ ์นญ ํ๋ค
-
์๊ฒฌ ์ถฉ๋์ 24์๊ฐ ๋ด์ ์ฌ๋ค๋ฆฌ ํ๊ธฐ๋ฅผ ํตํด ์กฐ์จ ํ๋ค
-
์๋ฌ ํธ๋ค๋ง์ ๋ฐฑ์๋ ํ๋ก ํธ ๋๋ค ์ฐธ์ฌ ํ๋ค
-
์ค๋ฒ์ปค๋ฎค๋์ผ์ด์ ์ ํตํด ์๊ฒฌ์กฐ์จ ๋ฐ ์ํฉ ๊ณต์ ํ๊ธฐ
-
ํ์คํ์ง ์์ ๋ถ๋ถ์ด ์๋ค๋ฉด ํผ์ ๊ฒฐ์ ํ์ง ๋ง๊ณ ์๋ก ๊ณต์ ํ๊ธฐ
-
๊ฐ์์ ์๊ฒฌ์ ์กด์คํ๋ฉฐ ๋ฐฐ๋ คํ๊ธฐ
-
Commit message
- ์ค ๋ฐ๊ฟ์ ํตํด์ ์ ๋ชฉ๊ณผ ๋ณธ๋ฌธ์ ๊ตฌ๋ถ
- ์ ๋ชฉ์ 50์๋ก, ๋ณธ๋ฌธ์ 72์๋ก ์ ํ
- ์ ๋ชฉ์ ๋๋ฌธ์๋ฅผ ์ฌ์ฉ
- ์ ๋ชฉ์ ๋ง์นจํ๋ก ๋๋ด์ง ์๊ธฐ
- ์ ๋ชฉ์ ๋ช ๋ น๋ฒ(imperative mode)์ ์ฌ์ฉ
- ์ค ๋ฐ๊ฟ์ ํตํด์ ์ ๋ชฉ๊ณผ ๋ณธ๋ฌธ์ ๊ตฌ๋ถ
- ์ ๋ชฉ์ 50์๋ก, ๋ณธ๋ฌธ์ 72์๋ก ์ ํ
- ์ ๋ชฉ์ ๋๋ฌธ์๋ฅผ ์ฌ์ฉ
- ์ ๋ชฉ์ ๋ง์นจํ๋ก ๋๋ด์ง ์๊ธฐ
- ์ ๋ชฉ์ ๋ช ๋ น๋ฒ(imperative mood)์ ์ฌ์ฉ
- ์ด๋ป๊ฒ ๋ณด๋ค๋ ๋ฌด์๊ณผ ์๋ก ์ค๋ช
- Commit Type Listve mood)์ ์ฌ์ฉ
- ์ด๋ป๊ฒ ๋ณด๋ค๋ ๋ฌด์๊ณผ ์๋ก ์ค๋ช
| ํ๊ทธ์ด๋ฆ | ์ค๋ช |
|---|---|
| Feat | ์๋ก์ด ๊ธฐ๋ฅ์ ์ถ๊ฐํ ๊ฒฝ์ฐ |
| Fix | ๋ฒ๊ทธ๋ฅผ ๊ณ ์น ๊ฒฝ์ฐ |
| Design | CSS ๋ฑ ์ฌ์ฉ์ UI ๋์์ธ ๋ณ๊ฒฝ |
| !BREAKING CHANGE | ์ปค๋ค๋ API ๋ณ๊ฒฝ์ ๊ฒฝ์ฐ |
| !HOTFIX | ๊ธํ๊ฒ ์น๋ช ์ ์ธ ๋ฒ๊ทธ๋ฅผ ๊ณ ์น๋ ๊ฒฝ์ฐ |
| Refact | ์ฝ๋ ๋ฆฌํฉํ ๋ง |
| Comment | ํ์ํ ์ฃผ์ ์ถ๊ฐ ๋ฐ ๋ณ๊ฒฝ |
| Docs | ๋ฌธ์๋ฅผ ์์ ํ ๊ฒฝ์ฐ |
| Chore | ๋น๋ ํ ์คํธ ์ ๋ฐ์ดํธ, ํจํค์ง ๋งค๋์ ๋ฅผ ์ค์ ํ๋ ๊ฒฝ์ฐ |
| Add | ํ์ผ ํน์ ํด๋๋ฅผ ์ถ๊ฐํ๋ ์์ ๋ง ์ํํ ๊ฒฝ์ฐ |
| Rename | ํ์ผ ํน์ ํด๋๋ช ์ ์์ ํ๊ฑฐ๋ ์ฎ๊ธฐ๋ ์์ ๋ง ํ๋ ๊ฒฝ์ฐ |
| Remove | ํ์ผ ํน์ ํด๋๋ฅผ ์ญ์ ํ๋ ์์ ๋ง ์ํํ ๊ฒฝ์ฐ |
| DevSave | commit ์ ์ค๊ฐ์ ์ฅ ํ ๊ฒฝ์ฐ |
| Test | ํ ์คํธ ์ฝ๋ ์ถ๊ฐ |
| Style | ์ฝ๋ ์๋ฏธ์ ์ํฅ์ ์ฃผ์ง ์๋ ๋ณ๊ฒฝ์ฌํญ |
Commit ์์
Add: [Client] #์ด์๋ฒํธ landing page
!BREAKING CHANGE: [Server] #5 change API arguments
ํ์ [Feat] ๋ก๊ทธ์ธ๊ธฐ๋ฅ ์์ค๋ก๊ทธ์ธ #5321 Closed
- semistandard ๋ฅผ ๋ฐ๋ฆ ๋๋ค.
์ฃผ์ ๊ท์น
-
๋ณ์์ด๋ฆ, ํ์ผ Camel-case ์ฌ์ฉ
-
DB & ์์ฑ์ ์ด๋ฆ Camel-case ์ฌ์ฉ
-
์ปดํฌ๋ํธ, ํ์ด์ง๋ Pascal-case ์ฌ์ฉ
-
space๋ ๋ ์นธ, ํญ ์ฌ์ฉ x
-
์ค์ง single quote๋ง ์ฌ์ฉ, ํ ํ๋ฆฟ ๋ฆฌํฐ๋ด์ ํํ์ ์ฌ์ฉํ ๋๋ง ์ฌ์ฉํ์ธ์.
-
var ๋ ์ฌ์ฉํ์ง ์์ต๋๋ค.
-
ํค์๋ ๋ค์์ ์คํ์ด์ค ํ๋๋ฅผ ๋์์ฃผ์ธ์.
-
ํ์์ ์ฝ๋ ์ดํด๋ฅผ ์ํด ์ ์ ํ ์ฃผ์ ์ฌ์ฉ ํด์ฃผ์ธ์
-
๋ ๋ง์ ์ ๋ณด๋ https://standardjs.com/rules.html ๋ฅผ ์ฐธ๊ณ ํ์ธ์.
-
- ์ ๋งํฌ์ ๋ฃฐ์์ ์ค์ง ํ ๊ฐ์ง๋ง ์์ธ์ ๋๋ค. ์ธ๋ฏธ ์ฝ๋ก ์ฌ์ฉ์ ํ์ฉํฉ๋๋ค.
-
PR
- ์๋ก์ด ๋ชจ๋ ์ค์น์ PR message์ ๋ฐ๋์ ๊ธฐ์ฌ
- PR ์ฌ๋ฆฌ๊ธฐ์ ์ branch ๋ฐ๋์ ํ์ธ
- PR ์ฌ๋ฆด ๊ฒฝ์ฐ ํ์๋ค์๊ฒ ๊ณต์
- PR ์ฌ๋ฆด ๋ issue ์นด๋ ๋งํฌํด์ ์์ฑ(https://bit.ly/2RbNlia)
- ํ์์๊ฒ ์ฝ๋ ๋ฆฌ๋ทฐ ๋ฐ์ ๋ค ๋ณธ์ธ์ด merge
- ์ํํ ์์ ๊ณผ ๋ถํ์ํ diff ์์ฑ์ ๋ฐฉ์งํ๊ธฐ ์ํด for, if, function ๋ฑ์์ ์ค๊ดํธ๋ ์๋ตํ์ง ์๊ธฐ
- ์๋ฏธ์๋ ๊ตฌ๋ณ์ ์ํด ๊ฒ์์ด ์๋๋ ์ด๋ฆ์ ์ง๊ณ ์ค์๋ง์ ํผํ๊ธฐ
-
branch
- Main branch์๋ merge ๊ธ์ง
- Feat/์ด์๋๋ฒ-๊ธฐ๋ฅ๋ช
| ์ข ๋ฅ | ์ฌ์ฉํจํด | ํน์ง |
|---|---|---|
| main | main | ํ๋ก๋์ ์ค๋ ์ท ๊ฐ์ฅ ์ต์ ์ ๋ฐฐํฌ๋ ๋ฒ์ |
| dev | dev | ๋ฆด๋ฆฌ์ฆ ๊ณํ์ ๋ฐ๋ผ์ Github์์ ๊ธฐ๋ณธ ๋ธ๋์น๋ก ์ง์ |
| feature | feature/์ด์๋ฒํธ-์ด๋ฆ feature/1-branch-name | dev์ ๋ณํฉ |
| hotfix | hotfix/์ด์๋ฒํธ hotfix/#911 | dev์ ์ ๋ณํฉ ํ ๋ฉ์ธ์ ๋ณํฉ |