Repo 작성자를위한 Git 풀 요청 에티켓 - 부적절한 스타일, 관련없는 변경 사항, 부분 수정

Question

GitHub에서 내 첫 풀 요청을 받았습니다. https://github.com/Ark-kun/Piranha/pull/1

의 변화는 매우 작고 이해하기 쉽다. 불행히도, 나는 너무 완벽합니다. 나는이 풀의 요청에 3 문제 참조 :

1. 깨진 스타일 (대신 공간 탭, 실종 중괄호)
2. 끌어 오기 요청이 상당히 관련이없는 수정이를 - 그것은 NullReferenceException를 고정하기위한 라이브러리 경로를 수정 Mac OS X.
3. 이 요청은 Mac OS X의 경로를 수정하지만 Linux에서는 수정하지 않습니다.

어떻게해야합니까?

1. 이 문제를 해결하기 위해 PR 저자에게 문의하십시오.
2. 코드를 직접 입력하고 수정하고 수락하지 않고 PR을 닫으십시오.
3. PR 분기를 체크 아웃하고 스타일과 코드를 수정하고 고정 분기를 병합하십시오. (아마도 커밋을 스쿼시)
4. 다른 것.

Answer 1

(1)과 (3)은 모두 합리적인 옵션입니다 (하지만 # 3을 선택한 경우 다른 작성자의 커밋을 스쿼시하지는 않습니다). 기회는 당신이 당신의 프로젝트에 개선 사항을 제출 사람들을 격려하고자하며, 저작자는 점에서 중요하다 - 내가 피할 것

(2)이 속성을 생략한다. 내 의견

Answer 2

첫 번째 옵션은, 다른 사람보다 더 나은 저장소 대중을하고 도움을하려고하는 사람들에게 신용을 제공하는 것이 좋다 그래서 풀 요청, 프로젝트에 도움 해야하는 필요하기 때문이다. 코드를 이식하고 홍보를 거절함으로써 자신의 공헌을 숨겨도 더 많은 일을하도록 권장하지 않습니다. 그래서 나는 분명히 옵션 2를 피할 것입니다. 둘째, 혼자서 코드를 수정하는 것은 시간 낭비 일 것입니다. 압니다를 수정하기위한 몇 가지 키보드 단축키 일 뿐이지 만 사람들이 표준에 부합하지 않는 코드 (공적 프로젝트와 일반적으로 너무 중요합니다)를 기여하게하면 앞으로 많은 수정이있을 것입니다.

Github에는 프로그래밍 사회를 하나로 모으기위한 많은 옵션이 있으며 PR 또는 코드 줄에 주석을 게시하는 것은 정말 쉽습니다. 그래서 당신이 당신의 기여자와 상호 작용할 때, 당신은 그를 더 나은 프로그래머로 만들고, 또한 그의 다음 PR은 더 나을 것입니다 (바라 건데 :)), 그래서 그것은 당신의 관심사에 있습니다. 때로는 재미있는 토론이 이러한 의견에서 시작되므로, 두 분 모두에게 경험을 배우는 것일 수도 있습니다.

물론 수정 사항이 실제로 작 으면 세 번째 "조치"를 선택하여 수정해야하지만 어쨌든 수정 사항에 대한 기여자에게 연락 할 것입니다.