Skip to content
New issue

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

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

Already on GitHub? Sign in 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 작성하기

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

No branches or pull requests

1 participant