-> ์๊ฒฉ ์ ์ฅ์์์ fork ๋ฒํผ์ ๋๋ฌ์ค๋ค.
-> "ํ๋ก์ ํธ ์ด๋ฆ"์ผ๋ก ๋ฐํํ๋ฉด์ ํ๋ก์ ํธ ๋ง๋ค๊ณ , "์๊ฒฉ ์ ์ฅ์ ๊นํ๋ธ ์ฃผ์"์ ๋ด์ฉ clone ํด์ด
-> rebase์ ์์ ๊ฐ์ ์ค๋ฅ๊ฐ ๋ฌ๋ค๋ฉด, ๋ด๊ฐ ์์
ํ๊ณ ์๋ ์ํ์์ ์๊ฒฉ ์ ์ฅ์์ ๋ด์ฉ ๊ฐ์ ธ์์ ์๊ฒจ๋๋ ์ค๋ฅ์ด๋ค. ๋ฐ๋ผ์, ์ผ๋จ, ๋ด๊ฐ ์์
ํ๊ณ ์๋ ๊ฒ์ ์คํ์ ๋ฃ์ด๋๊ณ , ์๊ฒฉ ์ ์ฅ์์ ๋ด์ฉ์ ๋จผ์ ๊ฐ์ ธ์จ ๋ค์์, ๋ด๊ฐ ํ๋ ์์
์ ๋ค์ ๊บผ๋ด์ค๋ฉด ๋๋ค.
-> fetch์ rebase๋ฅผ ํ๊ฑฐ๋ pull์ ํด๋ ๋๋ค. ๋๋ fetch์ rebase๋ฅผ ์ด์ฉํ๋ ๋ฐฉ๋ฒ์ผ๋ก ํ์
ํ์๋ค.
โ๏ธ ํด๊ฒฐ ๋ฐฉ๋ฒ:
-> ํ์ฅ์ด๋ผ๋ฉด, npx create-react-app ํด์ push ํด๋๊ธฐ
13.1. ํ์๋ค์ด pr ํ๋ค๊ณ ํ๋ฉด, ์๊ฒฉ ์ ์ฅ์์ pull-request์ 'file chages'์ ์ฝ๋ ๋ฆฌ๋ทฐํด์ค ํ, ์๋ฆฌ๊ธฐ
: ํด๋น repo๋ก ๋ค์ด๊ฐ์, Settings -> Manage access -> Invite a collaborator
: username์ด๋ email๋ฑ์ ์ด์ฉํ์ฌ ์ด๋ 9์ด๋๋ฐ์ ํ์๋ค์ ๋ฉ์ผ์ด๋ ๊นํ๋ธ ๋ฉ์์ง ํ๋ฉด์์ ํ์ธํ์ฌ accept ๋๋ฅด๋ฉด ๋จ)
: Looks Good To Me
-> ์ฝ๋์ ๋ณ ๋ฌธ์ ๊ฐ ์๋ค.
: as far as I know
-> ๋ด๊ฐ ์๊ธฐ์๋, ๋ด๊ฐ ๊ธฐ์ตํ๋ ํ
: in my opinion
-> ๋ด ์๊ฐ์๋
: in my humble opinion
-> ์ ๊ฒฌํด์๋ ์ (IMO๋ณด๋ค ์กฐ๊ธ ๋ ๊ฒฉ์ ์๋ ํํ)
: Sounds Good to Me
-> ์ข๋ค. 1๋ฒ์ด๋ ๊ฐ์ ๋ป
: Please Take Another Look
-> ์ ๋ฐ ์ข ๋ด์ฃผ์ธ์. ๋ฆฌ๋ทฐ์ด๊ฐ ๋ฆฌ๋ทฐ๋ฅผ ํด์ฃผ๋ค๊ฐ ์ค๊ฐ์ stop๋ ์ํ์์ ๋ฆฌ๋ทฐ์ข ํด๋ฌ๋ผ๊ณ ๋ถํํ ๋ ์ฐ๋ ๊ฒ
: For Your Information
-> ์ฐธ๊ณ ๋ก
: as soon as possible
-> ๊ฐ๋ฅํ ๋นจ๋ฆฌ, ์ต๋ํ ๋น ๋ฅด๊ฒ
: In Other Words
-> ๋ค๋ฅธ ํํ์ผ๋ก, ๋ค์ ๋งํ๋ฉด, ๋ฐ๊พธ์ด ๋งํ๋ฉด
: Too Long. Didn't Read
-> ๋ณดํต ๋ฌธ์ฅ ์์ ๊ธด ๊ธ์ ์์ฝํ ๋
: ๋ธ๋์น ์์ฑ => ์์ => PR => ์ฝ๋ ๋ฆฌ๋ทฐ => merge
vs code: ์ปดํจํฐ์ ๋๋ ํ ๋ฆฌ ์์ฑ ํ, ์๊ฒฉ ์ ์ฅ์ ์ฃผ์๋ฅผ ๋ณต์ฌํ์ฌ, F1 ์ ๋๋ฅด๊ณ git clone์ ์์ฑํ ํ url ๋ถ์ฌ๋ฃ๊ธฐ
: ๋ณดํต main ๋ธ๋์น์์ ์๋ก์ด ๋ธ๋์น๋ฅผ ๋ฐ๊ณ , ์ฌ๊ธฐ์ ์์
ํ ํ Pull Request(์ดํ PR)๋ฅผ ๋ ๋ ค ์ฝ๋๋ฆฌ๋ทฐ ํ merge
-> ๋ฉ์ธ์ด ๊ณ์ ๋ฐ๋๋ฉด ๋ค๋ฅธ ์์
์๋ ์๊ฒฉ๊ณผ ๋ค๋ฅธ ํ๊ฒฝ์์ ๊ฐ๋ฐ์ ์งํํ๊ฒ ๋์ด ์ ๋ง์ ์ถฉ๋์ ๋ง๊ฒ ๋ ๊ฒ ์ด๋ฉฐ
์์ฒญ๋ ๋นํจ์จ
: ๋ณ๊ฒฝ ์ฌํญ ๊ฐ์ ธ์ค๊ธฐ (์ฒซ commit ์๋๋ผ๋ฉด)
-> ๊ทธ๋ฌ๋ฉด, ์๋ก์ด ๋ธ๋์น๊ฐ ์์ฑ๋๊ณ , ๊ทธ๊ณณ์ ์์ ๋ ๋ด์ฉ์ด ๋ฐ์๋๋ค.
-> ํด๋น ๋ธ๋์น ์ด๋ฆ์ ๋๋ฅด๋ฉด, pull-request ํ๊ฒ ๋๋ ๋ฉ์์ง
: ์์ base์ ์ํ๋ ๋ธ๋์น๊ฐ ๋ฃ์ด์ ธ ์๋์ง ํ์ธํ ํ, ์ค๋ฅธ์ชฝ์ ํ์์ ๋ฆฌ๋ทฐ์ด๋ก ์ ์ ํ๊ณ , ์ ์ฑ์ค๋ฝ๊ฒ PR์ ์์ฑํ๊ณ , Create pull request
-> ๊ทธ๋ผ ์ด์ ๊ฐ์ ํ๋ฉด
-> PR ๋ ๋ฆฐ ํ, ๋ค๋ฅธ ํ์๋ค์๊ฒ ์ฝ๋ ๋ฆฌ๋ทฐ ์์ฒญํ๊ธฐ
------- ๋ ๋ค๋ฅธ ํ์์ ------
: Files changed๋ก ๊ฐ์ ๋ฆฌ๋ทฐ๋ฅผ ํด์ฃผ๊ณ
: ๋ชจ๋ ๋ฆฌ๋ทฐ๋ฅผ ๋๋๋ค๋ฉด ์ฐ์ธก ์๋จ์ Finish your review๋ฅผ ๋๋ฅด๊ณ Approve๋ฅผ ๋๋ฅด๊ณ Submit review๋ฅผ ๋๋ฅด๋ฉด ๋ฆฌ๋ทฐ ์๋ฃ
---------------- ๋ค์ ๋๋ ------
-> Reviewers์ ๋ฆฌ๋ทฐ๊ฐ ๋๋๋ฉด(Approve๊ฐ ๋๋ฉด) ์ค๋ฅธ์ชฝ ์๋จ์ฒ๋ผ ์ฒดํฌํ์
-> (merge๋ ๋ชจ๋ ํ์๋ค์ด ํ ์ ์์ง๋ง, ๋ณดํต ๋ณธ์ธ์ด ์ฑ
์์ง๋ค๋ ๋ง์ธ๋๋ก PR์ ์ฌ๋ฆฐ ๋ณธ์ธ์ด Mergeํ๋ค.)
-> merge ์ดํ ๋ค๋ฅธ ํ์๋ค์๊ฒ pull ์์ฒญ
: ๋ฉ์ธ ๋ธ๋์น์์ ๋ณ๊ฒฝ ์ฌํญ ํ์ธ ๊ฐ๋ฅ
โ๏ธ ์ ๋ฆฌ:
ํ๋์ ์๊ฒฉ ์ ์ฅ์์ ํ์
์ ํ๊ธฐ ์ํด์๋ ํ์๋ค์ด ๋ชจ๋ ๋ฉ์ธ ์๊ฒฉ ์ ์ฅ์๋ฅผ fork ํด์ค๊ณ , pull๋ก ๋ณ๊ฒฝ ๋ด์ฉ ๋น๊ฒจ์จ ์ดํ, ๊ฐ์ ๋ก์ปฌ์ ๋ธ๋์น ์๋ก ํ์ ์ฝ๋ ์์ ํ, ๊ฐ์์ ๋ก์ปฌ์์ add, commit, push ์ดํ, remote๋ก ์ฐ๊ฒฐ๋ ์๊ฒฉ ์ ์ฅ์์ PR์ ๋ณด๋ธ๋ค. ๊ทธ๋ผ ๋ค๋ฅธ ํ์๋ค์ ์ด๋ฅผ ํ์ธํ์ฌ code review๋ฅผ ๋จ๊ฒจ์ฃผ๊ณ , ๊ทธ์ ๋ฐ๋ผ merge๋ฅผ ํด์ค๋ค.
์ด๋ฐ ์ ์ฉํ ์ ๋ณด๋ฅผ ๋๋ ์ฃผ์ ์ ๊ฐ์ฌํฉ๋๋ค.