Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[조사 미션 수행] 좋은 PR이란? #5

Open
SeonHwan-Kim opened this issue Sep 2, 2022 · 0 comments
Open

[조사 미션 수행] 좋은 PR이란? #5

SeonHwan-Kim opened this issue Sep 2, 2022 · 0 comments

Comments

@SeonHwan-Kim
Copy link
Collaborator

좋은 PR은 무엇인가

PR의 규모를 적당히 가져가기

  • PR의 규모가 너무 크면 reviewer와 reviewee 모두가 힘들어진다
  • 보통 1개의 PR이 1000줄을 넘지 않게 정함

Reviewer 배려하기

  • 코드 컨벤션 잘 지키기.
  • 리뷰 가이드라인 잘 작성하기
  • 작업 중 리뷰 가능 여부 잘 명시하기

내가 쓴 코드 책임지기

  • 자신이 작성한 코드에 대해 100%이해하기
  • 무엇을 알고 무엇을 모르는지를 명확하게 하기

반드시 테스트 하기

  • 요청한 코드는 모두 정상적으로 작동해야한다.

자신의 예상 작업량 잘 측정하기

  • 잘 재고, 잘 쪼개기
  • 이슈 베이스 브랜치 활용하기

유용한 PR의 양식

# 주요 변경사항
	- 이 PR이 해결하는 문제
# 링크
	- 디자인 시안 링크 / 슬랙 관련 대화 링크
# 시급한 정도
	- 보통
	- 긴급
	- 천천히
# 중점적으로 봐줬으면 하는 부분
	- 변경사항이 큰 경우 집중해야 할 부분, 또는 불안해서 봐주었으면 하는 부분 등

출처
좋은 PR 작성하기

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant