search

ChanhuiSeok / chanhuiseok.github.io

GitHub Page 호스팅을 이용한 블로그입니다.
https://chanhuiseok.github.io/
MIT License
4 stars 3 forks source link

posts/git-3/ #6

Open utterances-bot opened 3 years ago

utterances-bot commented 3 years ago

[GitHub] Pull Request(PR) 보내는 방법 : Contribution 하기 | ChanBLOG

컴퓨터/IT/알고리즘 정리 블로그

https://chanhuiseok.github.io/posts/git-3/

connieya commented 3 years ago

안녕하세요, 글 잘 읽었습니다. 한 가지 궁금한 것이 있는데요 PR 승인 후 branch를 삭제 하는 특별한 이유가 있나요? 수정할 것이 생기면 또 branch를 만들고 PR를 해야하는데, 기존 branch에서 진행하면 안되나요?

ChanhuiSeok commented 3 years ago

@gunny6026 보통 pr 작성을 하면, 다른 사람들로부터 코드리뷰를 받아서 해당 작업에 대한 수정사항을 모두 반영한 후에 머지하게 됩니다. 그래서 해당 브랜치에서 계속 수정을 한 뒤, 최종 승인 및 머지될 경우 그 pr은 종료되었다고 봅니다.

그래서 반드시 그래야 하는 규칙은 아니지만, 보통 이미 끝난 브랜치를 재활용하는 편은 아니더라구요..!

만약 이미 머지가 끝난 브랜치가 남아있으면 불필요한 브랜치가 계속 쌓여서 관리 측면에서도 좋지 않을 수 있습니다.

이에 대해서 참고하시면 좋은 글이 있어서 링크를 남기도록 하겠습니다 :) https://ardalis.com/why-delete-old-git-branches/

hayoung-jeremy commented 3 years ago

친절한 설명 감사드립니다! 프로젝트 개발과정 기록중인데 참고하여 작성해도 될까요? 해당 페이지와 출처는 참조에 모두 작성할 예정입니다. 감사합니다 :)

ChanhuiSeok commented 3 years ago

@hayoung-jeremy 넵넵 마음껏 활용해 주세요!! :)

Hwan-I commented 3 years ago

설명 보고 바로 이해해서 PR까지 잘 수행할 수 있었습니다. 정말 감사 드리고 저도 혹시 이 글 참조해서 프로젝트 수행 때 참조 글로 써도 괜찮을까요? 윗분처럼 페이지와 출처는 참조에 모두 작성하려 합니다.

ChanhuiSeok commented 3 years ago

@Hwan-I 넵넵 물론입니다!!

Hwan-I commented 3 years ago

@ChanhuiSeok 네! 감사합니다!

yourimiyi commented 2 years ago

이해하기 쉬운 좋은 글 감사합니다. Git PR 이해가 한번에 되었습니다.

ChanhuiSeok commented 2 years ago

@jaehyung0 pr용으로 따로 만든 브랜치에 push를 하신 것이 맞을까요?? 지금 테스트해 보니 정상적으로 버튼이 나오고 있는데, 문제가 발생하고 있는 상황을 좀 더 알려주시면 감사하겠습니다.

keimindev commented 1 year ago

오픈소스에 기여하고 싶었는데 fork, 브랜치 생성 등 기존에 늘 해오던 pr과는 좀 달라서 헤맸는데, 이 글 덕분에 이해가 퐉퐉!! 감사합니다!! :)