42-CPP-GIT / CPP_GIT

Git Code Review๋กœ CPP Module๋ฅผ ๋•Œ๋ ค์žก์ž โ˜„๏ธ
9 stars 1 forks source link

๐Ÿ”ฅ CPP_GIT ๐Ÿ”ฅ


๐Ÿ’ช ์Šคํ„ฐ๋”” ๋ชฉํ‘œ

CPP์™€ GIT์„ ๊ฐ™์ด ํ•จ๊ป˜ ๋ฐฐ์šฐ์ž!

๐Ÿ“… ์Šคํ„ฐ๋”” ๊ธฐํ•œ

11์›” 15์ผ๊นŒ์ง€!

๐Ÿค Ground Rule

  1. C์–ธ์–ด La Piscine์ฒ˜๋Ÿผ ๊ฐ์ž ์ฝ”๋“œ๋กœ ์ง„ํ–‰์„ ํ•˜๊ธฐ! (๊ฐ์ž ์ด๋ฆ„์œผ๋กœ ๋œ Branch ์—์„œ ์ž‘์„ฑ)
  2. ํ•œ ํŒŒํŠธ๋ฅผ ๋‹ค ํ–ˆ์„ ๊ฒฝ์šฐ, ์˜คํ”„๋ผ์ธ์œผ๋กœ ํŒ€์›์—๊ฒŒ ํ‰๊ฐ€๋ฅผ ๋ฐ›๊ธฐ (์ตœ๋Œ€ 2๋ช… ๊ฐ€๋Šฅ!)
  3. ํ‰๊ฐ€ OK๋ฅผ ๋ฐ›์„ ๊ฒฝ์šฐ, PR์„ ๋‚ ๋ฆฌ๊ธฐ
  4. PR์„ ๋‚ ๋ฆด ๊ฒฝ์šฐ, ์ตœ๋Œ€ํ•œ ๋ณธ์ธ์ด ๋ฐฐ์šด ๊ฒƒ๋“ค ์ •๋ฆฌํ•ด์„œ ์ž‘์„ฑํ•˜๊ธฐ
  5. MERGE๋Š” ์ตœ์†Œ ์Šคํ„ฐ๋””์› 3๋ช…์ด Code Review๋ฅผ ํ•œ ํ›„, APPROVE๋ฅผ ํ•ด์•ผ ๊ฐ€๋Šฅํ•˜๋‹ค.
  6. ์ถœ์ฒ˜๋Š” ๊ผญ ํ‘œ์‹œ

โ‰๏ธ FAQ

Q. ์‹ค์ œ 42์„œ์šธ ํ‰๊ฐ€๋Š” ์–ธ์ œ ๋ฐ›์„ ์ˆ˜ ์žˆ๋‚˜์š”?
A. ์˜คํ”„๋ผ์ธ์œผ๋กœ ํŒ€์›์—๊ฒŒ ํ‰๊ฐ€ ๋ฐ›์œผ๋ฉด ๊ทธ ๋’ค ๋ถ€ํ„ฐ๋Š” ๊ฐ€๋Šฅํ•ฉ๋‹ˆ๋‹ค. (PR ์˜ฌ๋ฆฌ๊ณ , ์ฝ”๋“œ๋ฆฌ๋ทฐ ๋ฐ›๊ณ  42์„œ์šธ ํ‰๊ฐ€ ๋ฐ›๋Š” ๊ฒƒ์„ ๊ถŒ์žฅํ•˜๊ธด ํ•ฉ๋‹ˆ๋‹ค.)

Q. PR์ด Merge(closed)๋˜์–ด์„œ Code Review๋ฅผ ํ•˜์ง€๋ชปํ–ˆ์–ด์š”. Review๋ฅผ ํ•˜๊ณ  ์‹ถ์œผ๋ฉด ์–ด๋–ป๊ฒŒ ํ•˜์ฃ ?
A. PR์ด Closed๋˜์–ด๋„ Review๋Š” ์—ฌ์ „ํžˆ ๋‹ฌ ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค. ๋Š์ž„์—†์ด ์งˆ๋ฌธํ•ด๋ณด์„ธ์š”!

Q. ISSUES๋Š” ์“ฐ์ง€ ์•Š๋‚˜์š”?
A. ๊ฐ Module์„ ํ’€ ๋•Œ, ์•Œ์•„๊ฐ”๋˜ ๊ฒƒ๋“ค์„ ISSUES์— ์ž‘์„ฑํ•˜๊ณ , PR์—๋Š” ISSUES๋ฒˆํ˜ธ๋ฅผ ์ ์–ด์„œ ์„ค๋ช…์„ ํ•˜์…”๋„ ์ข‹์Šต๋‹ˆ๋‹ค.

Q. PR์„ ํ–ˆ๊ณ , MERGE๋Š” ์•„์ง ์•ˆํ–ˆ๋Š”๋ฐ ์ˆ˜์ •์‚ฌํ•ญ์ด ์ƒ๊ฒผ์–ด์š”. ํ˜น์€ ํ”ผ๋“œ๋ฐฑ์„ ๋ฐ›๊ณ  ์ˆ˜์ •ํ•˜๊ณ  ์‹ถ์–ด์š”. ๋‹ค์‹œ PR์„ ์˜ฌ๋ ค์•ผํ•˜๋‚˜์š”?
A. ์•„๋‹™๋‹ˆ๋‹ค. ๊ทธ Branch์—์„œ ๋‹ค์‹œ ์ฝ”๋“œ๋ฅผ ์ˆ˜์ •ํ•˜๊ณ , commit->push๋ฅผ ํ•˜๋ฉด ๊ทธ PR ์ฐฝ ํ•˜๋‹จ์— ๋ฐ”๋€ ์ •๋ณด๋“ค์ด ๋œจ๊ณ , ์ฝ”๋“œ๊ฐ€ ๋ฐ”๊ปด์„œ ์˜ฌ๋ผ๊ฐ‘๋‹ˆ๋‹ค. ๋‹ค์‹œ PR ์˜ฌ๋ฆฌ์‹ค ํ•„์š” ์—†์Šต๋‹ˆ๋‹ค.

Q. PR์„ ํ•ด์„œ Merge๋ฅผ ๋๋ƒˆ๋Š”๋ฐ ์ˆ˜์ •ํ•˜๊ณ  ์‹ถ์€ ์‚ฌํ•ญ๋“ค์ด ์žˆ์–ด์š”.
A. ์ด๋Ÿด ๋•Œ๋Š” ๋‹ค์‹œ PR ์˜ฌ๋ ค์ฃผ์…”๋„ ์ข‹์Šต๋‹ˆ๋‹ค.

๐Ÿงšโ€โ™€๏ธ PR ์‹œ, ์ž‘๊ณ  ๊ท€์—ฌ์šด ๋ฃฐ

Screen Shot 2022-10-24 at 8 57 44 PM

๋“ค์–ด์˜ค๋ฉด ์•„์‹œ๊ฒ ์ง€๋งŒ PRํ•  ๋•Œ ํ…œํ”Œ๋ฆฟ ์ถ”๊ฐ€ํ•ด๋†จ์Šต๋‹ˆ๋‹ค.

PRํ•  ์‹œ, ์ œ๋ชฉ์€ [Intra ID] CPP0X ์ ์–ด์ฃผ์‹œ๋ฉด ๋  ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค.
Reviewers๋Š” ์ž๋™์œผ๋กœ 3๋ช…์œผ๋กœ ์ ์šฉ๋  ์˜ˆ์ •์ž…๋‹ˆ๋‹ค.
Assignees๋Š” ๋ณธ์ธ์„ ํ•ด๋†“์œผ์‹œ๋ฉด ๋ฉ๋‹ˆ๋‹ค.

PR ํ…œํ”Œ๋ฆฟ์„ ๋ณด๋ฉด, PR์€ ์˜คํ”„๋ผ์ธ ํ‰๊ฐ€๋ฅผ 1ํšŒ ๋ฐ›์€ ๋’ค ์˜ฌ๋ ค์•ผํ•˜๋‹ˆ ํ‰๊ฐ€ํ•ด์ฃผ์‹  ๋ถ„ ์ธํŠธ๋ผ id๋ฅผ ์ ์–ด์ฃผ์‹œ๋ฉด ๋ฉ๋‹ˆ๋‹ค.
์ด๋ฒˆ ๊ณผ์ œ์— ์•Œ์•„์•ผํ•˜๋Š” ๊ฐœ๋…๋“ค์€ ์งง๊ฒŒ ์–ด๋–ค ๊ฒƒ๋“ค์ด ์“ฐ์˜€๋Š” ์ง€ ์ ์–ด์ฃผ์‹œ๋ฉด ๋ฉ๋‹ˆ๋‹ค.
์ด๋ฒˆ ๊ณผ์ œ๋ฅผ ํ†ตํ•ด ์•Œ์•„๋‚ธ ๊ฒƒ๋“ค์€ ๋ณธ๊ฒฉ์ ์œผ๋กœ ์–ด๋–ค ๊ฒƒ์„ ๋ฐฐ์šฐ์…จ๋Š” ์ง€ ์ ์–ด์ฃผ์‹œ๋ฉด ๋ฉ๋‹ˆ๋‹ค.

๋งฅ์„ ์“ฐ์‹œ๊ธฐ์— ์Šค์ƒท, ํ™”๋ฉด ๋…นํ™” ๋˜ํ•œ ์‰ฌ์šธํ…Œ๋‹ˆ ์ด๋Ÿฐ ๊ฒƒ๋“ค๋„ ์ ์–ด์ฃผ์‹œ๋ฉด ์•Œ๊ธฐ ์‰ฌ์šธ ๊ฒƒ ์ž…๋‹ˆ๋‹ค.
์ˆ˜์ •์‚ฌํ•ญ(optional)์€ pr์„ ์˜ฌ๋ ธ์Œ์—๋„ ๋ญ”๊ฐ€ ์ˆ˜์ •ํ•˜๊ณ  ์‹ถ์€ ๊ฒƒ์ด ์ƒ๊ฒผ๋‹คํ•˜์‹œ๋ฉด ์˜ฌ๋ฆฌ์‹œ๋ฉด ๋ฉ๋‹ˆ๋‹ค.
๊ฐ„๋‹จํ•œ ์ด์•ผ๊ธฐ์ด์ง€๋งŒ PR์„ ์˜ฌ๋ฆฌ๋ฉด ํ†ก๋ฐฉ์— ํ•œ๋ฒˆ ์ด์•ผ๊ธฐ๋ฅผ ํ•ด์ฃผ์‹œ๋ฉด ๋น ๋ฅด๊ฒŒ ๋ฆฌ๋ทฐํ•  ์ˆ˜ ์žˆ์„ ๊ฒƒ ์ž…๋‹ˆ๋‹ค.

๐Ÿ‘จโ€๐Ÿ‘ฉโ€๐Ÿ‘งโ€๐Ÿ‘ฆ ์Šคํ„ฐ๋”” ๋ฉค๋ฒ„

์žฅํ˜ธ์ง„ ํ”„๋กœํ•„
์žฅํ˜ธ์ง„(hchang)
๊น€๋ฏผ์ˆ˜ ํ”„๋กœํ•„ ๋ฐ•์ง„์˜ ํ”„๋กœํ•„ ์‹ฌ์„ฑ๋ฏผ ํ”„๋กœํ•„
๊น€๋ฏผ์ˆ˜(minsuki2) ๋ฐ•์ง„์˜(jinypark) ์‹ฌ์„ฑ๋ฏผ(sesim)
์ด์Šน์ˆ˜ ํ”„๋กœํ•„ ์˜ค์ •๋ฏผ ํ”„๋กœํ•„ ์ „์ค€ํ˜ธ ํ”„๋กœํ•„
์ด์Šน์ˆ˜(seungsle) ์˜ค์ •๋ฏผ (junoh) ์ „์ค€ํ˜ธ(junhjeon)