-
Notifications
You must be signed in to change notification settings - Fork 0
๐ ๊น ์ปจ๋ฒค์
YunHak Lee edited this page May 17, 2024
·
1 revision
๋ณต์กํ ๊ด๋ฆฌ ์ ๋ต์ ํ์์ฑ์ด ์ ๊ณ ๋น ๋ฅธ ์์ฑ์ ๋ชฉํ๋ก ๊น ๊ด๋ฆฌ์ ์ ์ ๋ฆฌ์์ค๋ฅผ ์ฐ๋ ๊ฒ์ด ๊ฐ๋ฐ์ ์ฉ์ดํ๋ฏ๋ก Github-Flow๋ฅผ ์ฑํํ๋ค.
- mainย : ๊ฐ๋ฐ์ด ์๋ฃ๋ ์ฐ์ถ๋ฌผ์ด ์ ์ฅ๋ ๊ณต๊ฐ -> ์ธ์ ๋ ์คํ๊ฐ๋ฅํ ์ํ์ฌ์ผ ํ๋ค.
- developย : ๊ฐ๋ฐ ์์ ์ด ์ด๋ฃจ์ด์ง๋ ๋ธ๋์น, ์ด์๋ณ/์์ ๋ณ๋ก ๋ธ๋์น๋ฅผ ์์ฑํ์ฌ ๊ธฐ๋ฅ์ ๊ฐ๋ฐํ๋ค
1. main ๋ธ๋์น์์์ ์์
์ ์์น์ ์ผ๋ก ๊ธ์งํ๋ค.
2. Pull Request๋ฅผ ์์ฑํ ํ ์นดํก๋ฐฉ์ ์ฝ๋๋ฆฌ๋ทฐ ์์ฒญ์ ์ฌ๋ฆฐ๋ค.
3. ์ฝ๋๋ฆฌ๋ทฐ๋ฅผ ํ๋ฉฐ ์๋ก์ ์ฝ๋๋ฅผ ์ดํดํ๊ณ ๊ณต๋ถํ๋ฉฐ ์ฑ์ฅํฉ์๋ค!
4. Merge์, ์ต์ main๋ฅผ ๋ณธ์ธ ๋ธ๋์น์ ๋จผ์ Mergeํ์ฌ ๋ก์ปฌ์์ Conflict๋ฅผ ํด๊ฒฐํ ๋ค, ์ฑ์ด ์ ์์ ์ผ๋ก ์คํ๋๋ ์ง ๋น๋ํ ํ Mergeํ๋ค.
<1 ์ด์> - <1 ๋ธ๋์น> - <n ๊ฐ์ PR> ์ ๋์ ๊ด๊ณ๋ก ์์ฑํ๋ค.
1. ํ๋์์ ์ฌ๋ฌ PR์ ํฌํจํ ์ ์๋ ์ ๋นํ ์์
๋จ์๋ก Issue๋ฅผ ์์ฑํ๋ค.
2. Miro์ ํด๋น ์ด์์ ์งํ์ํฉ์ ์ถ๊ฐํ๋ค.
3. Issue์ ๋์๋๋ ๋ธ๋์น๋ฅผ ํ๋ ์์ฑํ๋ค.
4. PR์ด ์ข
๋ฃ๋์ด๋ branch๋ ๋ฐ๋ก ์ญ์ ํ์ง ์์ผ๋ฉฐ, Issue๊ฐ ์ข
๋ฃ๋ ๋ branch๋ฅผ ์ญ์ ํ๋ค.
1. Reviewer
- ์จํํ ๋์์ค๋ก ํผ๋๋ฐฑ์ ๋จ๋ค.
- ์ ํจํ ๋ฆฌ๋ทฐ๊ฐ ๋ ์ ์๋๋ก ์ผ๋์ ๋๊ณ ๋ฆฌ๋ทฐํ๋ค.
2. ์ปค๋ฎค๋์ผ์ด์ ๋น์ฉ์ ์ค์ด๊ธฐ ์ํ Princess N ๋ฃฐ
P1: ๊ผญ ๋ฐ์ํด์ฃผ์ธ์ (Comment)
P2: ๋ฐ์ํด๋ ์ข๊ณ ๋์ด๊ฐ๋ ์ข์ต๋๋ค (Approve)
P3: ๊ทธ๋ฅ ์ฌ์ํ ์๊ฒฌ์
๋๋ค (Approve)
reference: https://github.com/DeveloperAcademy-POSTECH/MacC-Team-EarthValley80/wiki
์กฐ์ค | ๋๊ธฐ | ์ ์ด๋ฏธ | ์๋ผ | ์ ๋์ค | ์ ์ธ |
---|---|---|---|---|---|
@Cindy7878 | @Naknakk | @TakeMos | @yejinkang1025 | @zenith0001 | @darongzzang |