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은 무엇인가? #7

Open
HI-JIN2 opened this issue Sep 3, 2022 · 0 comments
Open

[조사 미션 수행] 좋은 PR은 무엇인가? #7

HI-JIN2 opened this issue Sep 3, 2022 · 0 comments

Comments

@HI-JIN2
Copy link
Collaborator

HI-JIN2 commented Sep 3, 2022

  • PR은 코드의 품질을 유지하기 위해 하는 것!
  • 코드 컨벤션을 잘 따를 것
  • 기능을 왜 추가하고 구현했는지 생각해볼 것
  • reviewer를 추가할 것
  • 내가 쓴 코드에 대해서 100 이해하고 있을 것
  • 적절히 쪼개서 PR을 올릴 것
💡 PR이 제때 merge되지 않으면 그 이후의 PR들이 합쳐져서 보이게 된다. 이를 해결하려면?

방법1. 이전 PR을 머지하고 머지이후 새롭게 브랜치를 생성한다.

방법 2. 첫번째 PR머지 이후 두번째 PR에서는 git rebase 하면 깔끔하게 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