-
Notifications
You must be signed in to change notification settings - Fork 0
Pull Request Guide
damiiya edited this page Sep 20, 2025
·
2 revisions
์ด ๋ฌธ์๋ SMOOTH ํ๋ก์ ํธ์ PR ์์ฑ ๊ท์น๊ณผ ํ
ํ๋ฆฟ์ ์ ๋ฆฌํ ๊ฐ์ด๋์
๋๋ค.
๋ชจ๋ ํ์์ PR ์์ฑ ์ ๋ณธ ๋ฌธ์๋ฅผ ์ฐธ๊ณ ํ์ฌ ์ผ๊ด์ฑ์ ์ ์งํด์ผ ํฉ๋๋ค.
-
ํ์:
# PR: T<์ํฝ.์ ์ค.ํ์คํฌ> - <๊ฐ๋จ ์์ฝ> โ US<๋ฒํธ> ๋จธ์ง -
์์:
# PR: T5.1.4 - ๋ฐ๊ฒฝ ๊ธฐ๋ฐ ์ ์ ํํฐ๋ง ๋ก์ง ๊ตฌํ โ US5.1 ๋จธ์ง
-
ํ์:
# PR: US<๋ฒํธ> - <๊ฐ๋จ ์์ฝ> โ dev ๋จธ์ง -
์์:
# PR: US5.1 - ์ค์๊ฐ ์ฌ๊ณ ์๋ฆผ ๊ธฐ๋ฅ ๊ตฌํ ์๋ฃ โ dev ๋จธ์ง
-
ํ์:
# PR: <prefix> - US<๋ฒํธ> <๊ฐ๋จ ์์ฝ># PR: <prefix> - dev <๊ฐ๋จ ์์ฝ>
-
prefix ๋ชฉ๋ก:
- feat: ์๋ก์ด ๊ธฐ๋ฅ ์ถ๊ฐ
- fix: ๋ฒ๊ทธ ์์
- refactor: ๊ธฐ๋ฅ ๋ณํ ์๋ ์ฝ๋ ๊ฐ์
- chore: ์ค์ /ํ๊ฒฝ/๋น๋ ๋ฑ ๋ถ์์ ์์
- test: ํ ์คํธ ์ฝ๋ ๊ด๋ จ ๋ณ๊ฒฝ
- hotfix: ๊ธด๊ธ ๋ฒ๊ทธ ์์
-
์์:
# PR: fix - US5.1 KST ํ์์คํฌํ ํ์ฑ ์ ์ฉ# PR: refactor - dev Redis ์ฃผ์ ๋ช ์ํ
PR ์์ฑ ์ ์๋ ํ ํ๋ฆฟ์ ๋ณต์ฌํด ์ฌ์ฉํฉ๋๋ค.
# PR: <๊ท์น์ ๋ง๋ ์ ๋ชฉ>
## ๋ชฉ์
- ์ด PR์ด ํ์ํ ์ด์ ๋๋ ๋ฐฐ๊ฒฝ์ ๊ฐ๋จํ ์์ฑํฉ๋๋ค.
- (์: KST ์๊ฐ ํ์ฑ ์ค๋ฅ ์์ , ์ค์๊ฐ ์๋ฆผ ๊ธฐ๋ฅ ๊ตฌํ, Redis ๊ตฌ์กฐ ๋ฆฌํฉํ ๋ง ๋ฑ)
---
## ๊ตฌํ/๋ณ๊ฒฝ ์ฌํญ
- <์ฃผ์ ๊ธฐ๋ฅ/์ฝ๋ ๋ณ๊ฒฝ 1์ค ์์ฝ>
- <์ฃผ์ ๊ธฐ๋ฅ/์ฝ๋ ๋ณ๊ฒฝ 1์ค ์์ฝ>
- (Task โ US ๋จธ์ง์ผ ๊ฒฝ์ฐ ํ์ Task ๋ฒํธ์ ์ค๋ช
์ ๋ชฉ๋กํ)
---
## ํ
์คํธ (์์ ๊ฒฝ์ฐ๋ง ์์ฑ)
- <ํ
์คํธ ํ๊ฒฝ> (์: Docker ๊ธฐ๋ฐ ๋ก์ปฌ ํ๊ฒฝ, staging ์๋ฒ ๋ฑ)
- <ํ
์คํธ ๋ฐฉ๋ฒ> (์: REST API ํธ์ถ, WebSocket ์ฐ๊ฒฐ, ๋จ์ ํ
์คํธ ์คํ ๋ฑ)
- <๊ฒ์ฆ ๊ฒฐ๊ณผ> (์: ๋ฉ์์ง ์์ ์ ์, ๋ฐ๊ฒฝ ํํฐ๋ง ์ฑ๊ณต, ์์ธ ์ฒ๋ฆฌ ํต๊ณผ ๋ฑ)
---
## BREAKING CHANGE (์์ ๊ฒฝ์ฐ๋ง ์์ฑ)
- <ํธํ์ฑ ๊นจ์ง / API ๋ณ๊ฒฝ / ํด๋ผ์ด์ธํธ ์์ ํ์ ์ฌํญ>
- (์: `/user/{userId}/alert` โ `/user/queue/alert` ๋ณ๊ฒฝ, timestamp ํฌ๋งท KST ํ์ ๋ฑ)
---
## ์ฐธ๊ณ
- ๊ด๋ จ ์ด์/์ ์ ์คํ ๋ฆฌ: Refs: US<๋ฒํธ> (๋๋ Task ๋ฒํธ)
- ๊ธฐํ ํ์ ์์
์ด๋ ์ฃผ์์ฌํญ
- (์: JWT ์ธ์ฆ์ ์ถํ ์ฐ๋ ์์ , ๋ก๊น
๋ ๋ฒจ์ ์์ ์ํฅ ์กฐ์ ๋ฑ)