Skip to content

πŸ“„μ½”λ“œ 리뷰 ν•˜λŠ” 방법

tgyuu-An edited this page Jan 1, 2024 · 8 revisions

Hits



πŸ“„Git μ„€μΉ˜λΆ€ν„° PRκΉŒμ§€ μ΄ν›„μ˜ λ‚΄μš©μž…λ‹ˆλ‹€.

μš”μ•½

  1. 리뷰 λŒ€κΈ° 쀑인 PR μ°ΎκΈ°
  2. 문제 링크둜 μ΄λ™ν•˜μ—¬ 문제 확인 및 μž‘μ„±μžμ˜ μˆ˜λ„ μ½”λ“œ 읽기
  3. Files changed νƒ­μœΌλ‘œ 이동
  4. μ½”λ“œ 리뷰 쀑 κΆκΈˆν•œ μ μ΄λ‚˜ μΉ­μ°¬ν•˜κ³  싢은 점, μƒˆλ‘­κ²Œ μ•Œκ²Œ 된 점이 μžˆλ‹€λ©΄ ν•΄λ‹Ή μ½”λ“œ λΈ”λŸ­μ— μ½”λ©˜νŠΈ 남기기
  5. Start a review 선택
  6. μ½”λ©˜νŠΈ μž‘μ„± ν›„ Review changes μ„ νƒν•˜μ—¬ 리뷰 μ™„λ£Œν•˜κΈ°





리뷰 κΈ°λ‹€λ¦¬λŠ” 쀑 πŸ”₯ 라벨이 뢙은 PR을 μž½μ‹Έκ²Œ μ°Ύμ•„μ€λ‹ˆλ‹€.






문제 링크에 λ“€μ–΄κ°€μ„œ μ–΄λ–€ λ¬Έμ œμΈμ§€ λΉ λ₯΄κ²Œ ν™•μΈν•œ λ’€, μž‘μ„±μžκ°€ μž‘μ„±ν•œ μˆ˜λ„ μ½”λ“œλ₯Ό 읽어본닀.






μˆ˜λ„ μ½”λ“œλ₯Ό λ‹€ μ½μ—ˆλ‹€λ©΄, μƒλ‹¨μœΌλ‘œ μ˜¬λΌμ™€μ„œ Files changed에 λ“€μ–΄κ°„λ‹€.






μ½”λ“œλ₯Ό 읽닀가, κΆκΈˆν•œ λΆ€λΆ„μ΄λ‚˜ μΉ­μ°¬ν•˜κ³  싢은 λΆ€λΆ„, μƒˆλ‘­κ²Œ μ–»μ–΄κ°€λŠ” 뢀뢄이 μžˆλ‹€λ©΄

μ•„λž˜ μ‚¬μ§„μ²˜λŸΌ λΉ¨κ°„ λ„€λͺ¨ λ°•μŠ€ μ•ˆμ— + λ²„νŠΌμ— λ“œλž˜κ·Έλ₯Ό ν•˜λ©΄ ν•΄λ‹Ή μ½”λ“œ λΈ”λŸ­μ— μ½”λ©˜νŠΈλ₯Ό 남길 수 μžˆμŠ΅λ‹ˆλ‹€.

μ½”λ“œ 리뷰 잘 ν•˜λŠ” 법을 참고해보면 쒋을 것 κ°™λ‹€...! 😊😊



이 λ•Œ, Add single comment κ°€ μ•„λ‹Œ, start a review둜 ν•΄μ€λ‹ˆλ‹€.

Add a single comment둜 ν•  경우 μ½”λ©˜νŠΈ ν•˜λ‚˜ν•˜λ‚˜ λ§ˆλ‹€ ν•΄λ‹Ή PRμž‘μ„±μžμ—κ²Œ μ•Œλ¦Όμ΄ κ°€μš”. πŸ₯²πŸ₯²πŸ₯²






μ½”λ“œμ— μ½”λ©˜νŠΈλ₯Ό λ‹€ λ‚¨κ²ΌμœΌλ©΄, 우츑 상단에 Review changesλ₯Ό λˆŒλŸ¬μ€λ‹ˆλ‹€.






λ§ˆμ§€λ§‰μœΌλ‘œ ν•΄λ‹Ή PR에 λŒ€ν•œ 마무리 멘트λ₯Ό λ‚ λ €μ€λ‹ˆλ‹€. 그리고 Approve에 체크 ν›„, submit Reviewλ₯Ό λˆŒλŸ¬μ€λ‹ˆλ‹€.


CommnetλŠ” ν•΄λ‹Ή PR에 ν”Όλ“œλ°±μ„ 쀄 λ•Œ μ‚¬μš©ν•©λ‹ˆλ‹€.

ApproveλŠ” ν•΄λ‹Ή 브랜치의 PR을 Main 브랜치둜 λ³‘ν•©ν•˜λŠ” 데 μŠΉμΈν•œλ‹€λŠ” 뜻 μž…λ‹ˆλ‹€.

Request ChangesλŠ” ν•΄λ‹Ή PRμ—μ„œ μˆ˜μ •ν•  뢀뢄이 μžˆμ„ λ•Œ, μˆ˜μ •μ„ μš”μ²­ν•˜λŠ” 뜻 μž…λ‹ˆλ‹€. 즉 이 것을 μ²΄ν¬ν•˜λ©΄ 병합을 μŠΉμΈν•˜μ§€ μ•ŠλŠ”λ‹€λŠ” λœ»μž…λ‹ˆλ‹€.


μ €ν¬λŠ” 단지 μŠ€ν„°λ””μ΄λ―€λ‘œ, Approveμ™Έμ—” λ”±νžˆ μ‚¬μš©ν•  게 μ—†μŠ΅λ‹ˆλ‹€.






Approveλ₯Ό ν–ˆλ‹€λ©΄, Reviewersμ—μ„œ μžμ‹ μ˜ 아이디 μ˜†μ— βœ”οΈλ‘œ λ°”λ€œμ„ λ³Ό 수 μžˆμŠ΅λ‹ˆλ‹€.






λ§ˆμ§€λ§‰μœΌλ‘œ, λ‚΄κ°€ ν•΄λ‹Ή PR의 λ§ˆμ§€λ§‰ 리뷰어라면 (λ‚˜λ₯Ό μ œμ™Έν•œ 2λͺ…이 λͺ¨λ‘ 리뷰λ₯Ό 남기고 λ‚΄κ°€ 3번째 라면) μƒλ‹¨μœΌλ‘œ μ˜¬λΌμ™€μ„œ,

μ½”λ“œ 리뷰 κΈ°λ‹€λ¦¬λŠ” 쀑πŸ”₯ 라벨을 리뷰 μ™„λ£Œ βœ”οΈ 라벨둜 λ°”κΏ”μ€λ‹ˆλ‹€.

이 λ•Œ μžκΈ°κ°€ 올린 PR이 μ•„λ‹ˆλΌλ©΄ μ ˆλŒ€λ‘œ λ§ˆμŒλŒ€λ‘œ Mergeν•˜μ§€ μ•ŠμŠ΅λ‹ˆλ‹€.






λ§Œμ•½, μžμ‹ μ˜ PR이 λ‹€λ₯Έ μŠ€ν„°λ””μ›μ˜ 리뷰λ₯Ό λͺ¨λ‘ λ°›μ•„μ„œ, 리뷰 μ™„λ£Œ βœ”οΈ 라벨이 λ˜μ—ˆλ‹€λ©΄,

μž½μ‹Έκ²Œ λ‚˜μ˜ PR둜 λ“€μ–΄κ°‘λ‹ˆλ‹€.






ν•΄λ‹Ή PR의 μ½”λ©˜νŠΈλ₯Ό ν™•μΈν•œ ν›„ 닡글이 ν•„μš”ν•œ μ½”λ©˜νŠΈλŠ” 닡글을 λ‹¬μ•„μ€λ‹ˆλ‹€.






λ‹¬λ§Œν•œ μ½”λ©˜νŠΈλ₯Ό λ‹€ λ‹¬μ•˜λ‹€λ©΄, 맨 μ•„λž˜λ‘œ λ‚΄λ €μ™€μ„œ Merge Pull Request λ₯Ό 눌러주면 끝 μž…λ‹ˆλ‹€.






μΆ”κ°€μ μœΌλ‘œ, Github 우츑 상단에 λ°•μŠ€ λͺ¨μ–‘ μ•Œλ¦Όν‘œμ‹œκ°€ 뜨면,

자주 자주 ν™•μΈν•΄μ€λ‹ˆλ‹€.

λ‚˜μ™€ κ΄€λ ¨λœ PR에 μ½”λ©˜νŠΈκ°€ 달린 것을 λ°”λ‘œ 확인할 수 μžˆμŠ΅λ‹ˆλ‹€..!







++μΆ”κ°€μ μœΌλ‘œ Mergeλ˜μ–΄ Closed된 μƒνƒœμ˜ PR도 μ½”λ©˜νŠΈλ₯Ό 남길 수 μžˆμŠ΅λ‹ˆλ‹€.

κ·ΈλŸ¬λ‹ˆ 라벨이 λ³€κ²½λ˜μ—ˆμœΌλ©΄ κ·Έλƒ₯ Mergeν•΄μ£Όμ‹œκ³ , λ‹«νžŒ PRμ—μ„œ 또 λŒ€ν™”λ₯Ό μ΄μ–΄λ‚˜κ°€μ‹œλ©΄ λ©λ‹ˆλ‹€.



κΈ΄ κΈ€ μ½μ–΄μ£Όμ…”μ„œ κ°μ‚¬ν•©λ‹ˆλ‹€. 😊😁😊😁

written by : tgyuuAn