-
Notifications
You must be signed in to change notification settings - Fork 5
๐ฅ ์คํฐ๋ Branch ๋ฐ PR ๊ท์น ๐ฅ
DONGMIN edited this page Oct 27, 2022
·
7 revisions
- repository fork & clone
- ์๋์ ํด๋ ๊ตฌ์กฐ ํญ๋ชฉ ์ฐธ๊ณ ํ์ฌ ํ์ผ ์์ฑ ํ ์์ค์ฝ๋ ์์ฑ
- ๋ณธ์ธ ์ด๋ฆ์ branch ์์ฑ
- commit(์ปค๋ฐ ๋ฉ์์ง๋
[์ถ์ฒ, ๋ฒํธ] ๋ฌธ์ ์ ๋ชฉ
์ผ๋ก ํต์ผ โ[PG, 1845] ํฐ์ผ๋ชฌ
) - ํ๋ก๊ทธ๋๋จธ์ค:
PG
, ๋ฐฑ์ค:BOJ
, ๋ฆฟ์ฝ๋:LC
- merge ํ Pull Request (PR ์ ๋ชฉ: N์ฃผ์ฐจ PR - ์ด๋ฆ, ๋ด์ฉ์ ๋ฌธ์ ๋ฅผ ์ด๋ค ์์ผ๋ก ํ์๋ ์ง์ ๋ํด์ ๊ฐ๋จํ๊ฒ ์ค๋ช ํ๋ค.)
PR ๋ฐฉ๋ฒ ์ฐธ๊ณ
- ์ ๋์ ์ผ๋ก ๋ง์ถฐ์ผ ํ๋ ๊ฒ์ ์๋๋ ์ตํต์ฑ ์๊ฒ ํ๋ฉด ๋๋ค.
์ด๋ฆ
/๋ฌธ์ ์ถ์ฒ
/๋ฌธ์ ์ ํor๋ ๋ฒจ
/๋ฌธ์ ์ถ์ฒ๋ฌธ์ ๋ฒํธ
- ๋ฌธ์ ํ์ผ ์ด๋ฆ์ ๋ณธ์ธ์ด ํธํ๋๋ก ์์ฑํ๋ค.
์์) dongmin/programmers/level1/PG1845
์์) dongmin/baekjoon/greedy/BOJ1234
- ์ ๋ชฉ์
N์ฃผ์ฐจ PR - ์ด๋ฆ
์ผ๋ก ์์ฑํ๋ค. - ๋ณธ๋ฌธ์๋ ๋ฌธ์ ๋ฅผ ํ๋ฉด์ ๋๊ผ๋ ์ ์ด๋ ์๋ก ์๊ฒ๋ ์ , ์ด๋ ค์ ๋ ์ ๋ฑ์ ์ ๋๋ค.
- ๋ฌธ์ ๋ฅผ ๋ชปํ์๋ค๋ฉด, ์ด๋ค ์์ผ๋ก ์ ๊ทผํ๋์ง์ ๋ํด์ ๊ฐ๋ตํ๊ฒ ์์ฑํ๋ค.
- ๋ผ๋ฒจ์ ์ฌ์ฉ์ธ์ด, ๋ฌธ์ ์ถ์ฒ ๋ฑ ์ ์ ํ ๋ผ๋ฒจ์ ์ฌ์ฉํ๋ค.
PR์ ์์์ผ ์ค์ 9์์ ๊น์ง ๋ฐ๋์ ์ ์ถ!!
- ์์ ๋กญ๊ฒ ๋ค๋ฅธ ์ฌ๋์ด PRํ ์ฝ๋๋ฅผ ๋ณด๊ณ ๋๋ ์ ์ด๋ ๋ณด์ํด์ผ ๋ ์ ๋ฑ์ ์ ๋๋ค.
- PR ๋ฆฌ๋ทฐ๋ฅผ ํ๋ฉด์ ๋ค๋ฅธ ์ฌ๋์ ์ฝ๋์์ ๊ถ๊ธํ๊ฑฐ๋ ์ดํด๊ฐ ์๋๋ ๋ถ๋ถ๋ค์ ์ ๋ฆฌํด์ ์ง๋ฌธํ๊ธฐ!
- ๋ชจ๋ ํ์๋ค์ ์ฝ๋ ๋ฆฌ๋ทฐ๊ฐ ์๋ฃ๋๋ฉด merge๋ฅผ ์งํํ๋ค.
- ๋ง์ฝ ๋ฆฌ๋ทฐ๋ฅผ ์ํ ํ์์ด ์๋ค๋ฉด ํด๋น ํ์์๊ฒ ๋ฆฌ๋ง์ธ๋ ํด์ค๋ค.