๐ข Code merge ๊ด๋ จ ๊ฑด
cyeji opened this issue ยท 1 comments
cyeji commented
ํ๋ก์ ํธ PR Merge ๊ด๋ จ ๊ฑด ์์ ์ฌํญ ์ ๋ฐ์ดํธ ํ์ต๋๋ค.
Before
์ผ๊ด merge
Now
Reviewer ์ ์ฒญ์๊ฐ ๋ชจ๋ ์น์ธ์ ํ์์์๋ง merge๊ฐ๋ฅ
cyeji commented
PR ํ Reviewer ๋ฑ๋ก ๋ฐฉ๋ฒ
- Create Pull Request ํ๋ฉด ์ค๋ฅธ์ชฝ Reviewers ์ค์ ๋ฐ Assignees ์ค์ (๋๋ช ์ ๋ถ ์ ํํ ๊ฒ)
- ์ ํ ์๋ฃ ์ ํ๊ธฐ๋๋ ํ๋ฉด
Reviewํ ์ํ ๋ณ๊ฒฝ
- ReviewChanges ๋ฒํผ ํด๋ฆญ ํ, ์ํ๋ณ๊ฒฝ (comment, approve, request changes)
- comment (์ฝ๋ฉํธ๋ง ๋ฌ๊ณ ๋จธ์ง ๋๋๋ก)
- approve (์น์ธ)
- requestChanges (ํผ๋๋ฐฑ ๋ฐ๊ณ ์์ ํ๊ธฐ ์ ๊น์ง merge ๊ธ์ง)
์ํ๋ณ๊ฒฝ ์ค์ ์ ์ ๋์ ์ผ๋ก ์ค์ ํฉ๋๋ค.
ex. ํฌ๊ฒ ๋ฌธ์ ๋ ๋์ง ์์ง๋ง ๊ณ ์น๋ฉด ์ข์ Comment, ์ฝ๋ ๋ณ๊ฒฝ ์์ฒญ ํ์ ์๋ ์ํ Approve, ์ฝ๋์์ ๋ณ๊ฒฝ๋์ง ์์ผ๋ฉด ๋ฌธ์ ๊ฐ ์์ ๋ requestChanges