π GitμΌλ‘ νμ μμνκΈ° #9 – 리뷰λ νμ§μ΄λ€ : 리뷰μ΄/μμ±μ μν κ³Ό νμ λ§€λ
πΉ 17. 리뷰μ΄/μμ±μ μν λΆλ΄ κΈ°μ€κ³Ό λ§€λ
π 17-1. PRμ "κ²ν λ°μ μ€λΉ"κ° λμμ λλ§ μμ±
π PRμ 리뷰μ΄μκ² λ³΄λ΄λ 건 κ²ν μμ²μ΄ μλ κ²ν μ±
μ μ΄κ΄
β‘οΈ λ―Έμμ±/μΆ©λ μλ PRμ νΌλλ§ μ λ°
βοΈ λ¦¬λ·° μ μ μλ 체ν¬λ¦¬μ€νΈ μλ£νκΈ° :
π git diffλ‘ λ³κ²½ λ΄μ μ€μ€λ‘ κ²ν
π λλ½λ νμΌ μλμ§ git status νμΈ
π μ»€λ° λ©μμ§ λ Όλ¦¬μ μ΄κ³ λͺ©μ λ³λ‘ ꡬμ±
π PR μ€λͺ μ κΈ°λ₯ μμ½, μ£Όμμ¬ν, μ μΈ νμΌ λͺ μ
π 17-2. 리뷰μ΄λ 'λ°©μ΄μ 리뷰'κ° μλλΌ 'νλ ₯μ 리뷰'
π λ¨μ μ€λ₯ μ°ΎκΈ°λ³΄λ€ κ°μ μ μκ³Ό λμ μ μκ° ν΅μ¬
β‘οΈ “μ΄ λΆλΆ λ¬Έμ μμ”λ³΄λ€ “μ΄λ° λ°©ν₯μ μ΄λ¨κΉμ?”κ° λ μμ°μ
βοΈ μ½λ 컨벀μ , ν μ€νΈ λ²μ, μ±λ₯ μν₯κΉμ§λ ν¨κ» κ³ λ €
π 17-3. μ½λμ λν΄ λ§νλ, μ¬λμ νκ°νμ§ μκΈ°
β “μ΄κ±° μ μ΄λ κ² νμ΄μ?”
β
“μ΄ λΆλΆμ μ΄λ° μλλ‘ μμ±νμ κ±ΈκΉμ?”
→ μλ νμ
→ λ¬Έμ 곡μ → κ°μ λ°©ν₯ ν©μ
π 17-4. 리뷰λ λΉ λ₯΄κ², νΌλλ°±μ λͺ
ννκ²
π 리뷰 μ§μ°μ κ°λ° νλ¦ μ 체λ₯Ό λ¦μΆ€
β‘οΈ ν루 λλ 리뷰 μ§μ°μ 곡μ λλ μλ¦Ό νμ
βοΈ νΌλλ°±μ μ λ§€ν νν λμ λͺ
ννκ²
μ :
β “μ΄κ±° μ’ μ΄μν κ² κ°μμ”
β “μ΄ λΆλΆμ null κ°λ₯μ±μ΄ μμΌλ Optional μ²λ¦¬ κ³ λ € λ°λλλ€”
π‘ μμ½ μ 리
1οΈβ£ PRμ μμ±λ μνμμλ§ μμ±
2οΈβ£ 리뷰λ μ§μ λ³΄λ€ μ μ μ€μ¬
3οΈβ£ ννμ μ½λ κΈ°μ€, μ¬λμ΄ μλ λ¬Έμ μ€μ¬
4οΈβ£ νΌλλ°±μ λͺ
ννκ³ λΉ λ₯΄κ²