«pull-requests» 태그된 질문

모든 심각한 Github 저장소는 왜 커밋을 단일 커밋으로 스쿼시하기를 원합니까? 나는 git log가 거기에 있다고 생각했기 때문에 모든 기록을 검사하고 어떤 변화가 어디서 발생했는지 정확하게 볼 수 있지만 그것을 스쿼시하면 기록에서 꺼내어 하나의 커밋으로 묶습니다. 요점이 뭐야? 이것은 또한 "미리 커밋하고 자주 저지르는"진언에 위배되는 것으로 보인다.

199 git  github  branching  pull-requests  etiquette 

이 질문은 Software Engineering Stack Exchange에서 답변을 얻을 수 있기 때문에 Software Quality Assurance & Testing Stack Exchange에서 마이그레이션 되었습니다. 작년에 이주 . 풀 요청의 변경 사항을 검토 할 때 종종 "TODO"메모가 포함 된 주석을 우연히 발견 할 수 있습니다. 문제를 해결하는 데 사용되는 솔루션은 개선 될 수 있지만 훨씬 …

24 code-quality  teamwork  code-reviews  management  pull-requests 

나는 Github에서 프로젝트를 포크하고 약간의 변경을 한 후 원래 관리자에게 가져 오기 요청을 보냈습니다 Merged pull request #11 from my_username/master. 나는했다 : 나는 확실하지 에티켓 지금 무엇을 해요, 그래서 이것은 내가이 일을하고있어 처음으로 git pull upstream master다음과 git push origin master, 이제 마지막으로 내 자신의 저장소 읽고 커밋 Merged pull …

17 git  github  pull-requests 

우리 회사의 여러 팀이 전에는 본 적이없는 코드 검토 워크 플로를 연습합니다. 회사 전체의 일관성을 유지하는 데 가치가 있다는 생각으로 그 배후의 생각을 이해하려고합니다. (여러 코드베이스에 기여하고 과거의 차이점으로 인해 넘어졌습니다.) 코드 작성자가 풀 요청을 제출 검토자가 코드를 검사 검토자가 승인하면 "좋아요, 자유롭게 병합하십시오"라는 문구를 따라 의견을 남깁니다. 검토자가 우려 …

16 code-reviews  pull-requests 

단순히 분기없이 마스터로 분기를 병합하는 대신 풀 요청을 사용하면 어떤 이점이 있습니까? 특히 모든 개발자가 마스터에 완전히 액세스 할 수있는 팀에서.

16 version-control  git  development-process  merging  pull-requests 

TFS에서 풀 요청에서 수정 될 사항에 대한 의견을 제출하기 전에 의견을 제출할 때이를 거부 또는 작성자 대기로 표시해야합니까? 어떤게 더 좋아?

13 team-foundation-server  pull-requests 

우리는 Bitbucket에서 GitHub로 전환하고 있으며 우리가 어려움을 겪고있는 한 가지는 다음과 같이 Bitbucket에서 매우 원활하게 작동하는 피어 코드 검토입니다. 작성자가 풀 요청을 열었습니다 (GitHub : 동일) 작성자가 동료를 검토 자로 추가했습니다 (GitHub : ?? 여러 담당자 와 함께 어려움을 겪고 있습니다 ). 검토 자 : 초록색 확인 표시가있는 PR 승인 …

12 code-reviews  github  pull-requests 

유용한 중앙 기능을 갖춘 github에서 훌륭한 프로젝트를 찾았지만 오류 처리, 로깅, 구성 및 설정의 "광택"에는 어려웠습니다. 이 프로젝트는 5 년 동안 진행되었으며 수백 줄의 코드 만 있습니다. 그래도 괜찮은 수의 감시자와 소수의 포크에 관심을 기울일만큼 유용합니다. 사용하기 위해 특정 추가 작업이 필요했지만 그 전에 정리 작업을 수행했습니다. 그런 다음 약간의 …

12 open-source  pull-requests  etiquette 

나는 지금 꽤 오랫동안 GitHub를 사용해 왔으며 일반적으로 기능 분기를 푸시 한 다음 직접 병합 요청을 시작하는 데 사용했습니다. 분기를 병합 한 위치를 추적하는 데 도움이되었습니다. 그러나 최근에 Git의 작동 방식에 대해 점점 더 많이 읽었으며 분기를 병합 할 때 병합 커밋을 사용하여 참조 할 수 있음을 깨달았습니다. 마스터에 기능이 …

12 git  github  merging  pull-requests 

마스터로의 Pull Request의 모든 코드는 프로덕션 준비가되어야한다는 개념이 있습니다. 이것은 의미가 있으며 내 의견으로는 공정한 진술입니다. 여기서 PR을 만들면이를 마스터에 넣었을 것이라는 의견이 있지만 일부 검토자가 단순히 '동의'하고 누락 된 부분을 발견하기를 원합니다. 우리는 인간이기 때문에 실수를 저지르고 다른 검토자가 단위 테스트에서 찾을 수없는 항목 (맞춤법 오류, 잘못된 javadoc 등)을 …

11 agile  pull-requests 

git을 주니어 팀 멤버 (협동 조합)에게 소개하고 있습니다. 그들은 추가, 커밋, 밀기 및 당기기의 기본에 익숙해졌습니다. 이제 요청과 지점을 가져 오기 위해 소개하겠습니다. 지점에서 풀 요청을 시작하면 진행중인 작업에 대해서도 동일한 작업을 수행해야합니까? 나는 그들의 풀 요청을 병합하는 사람이 될 것입니다. 나는 내가 지점에서 일하는 것이 가장 합리적 일지 확신하지 …

11 git  github  gitflow  pull-requests 

일주일 전, 나는 LFNW 에 있었고 래리 헤이스팅스와 대화를 나눈 후 다음과 같이 말했습니다. Git에는 Mercurial에는없는 풀 요청 워크 플로우를 가능하게하는 것이 있습니다. 이것이 BitBucket의 Pull Request가 좋지 않은 이유입니다. (문의를 위해, 우리는 파일-버그-그럼-첨부-패치 워크 플로우와 반대로, PR 워크 플로우 때문에 파이썬이 Mercurial에서 Git로 이동하고 있다는 사실을 논의하고있었습니다.) 그는 여기서 …

10 git  mercurial  pull-requests  bitbucket