수정해야 할 사항이있을 때 TFS에서 거부 또는 작성자 대기를 사용해야합니까?


13

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

여기에 이미지 설명을 입력하십시오

답변:


15

풀 요청이있는 검토 코드 에 대한 Microsoft의 내용 : 변경에 대한 투표 는 각 승인 등급의 제안 된 목적은 다음과 같습니다.

제안 사항 승인 : 풀 요청에 동의하지만 코드를 개선하기위한 제안 사항 (선택 사항)을 제공하십시오.

작성자 대기 : 변경 사항을 승인하지 말고 작성자에게 의견을 검토하도록 요청하십시오. 저자는 문제를 해결 한 후 언제 코드를 다시 검토해야하는지 알려야합니다.

거부 됨 : 변경 사항이 허용되지 않습니다. 이 방법으로 투표하는 경우 변경 요청이 거부 된 이유를 자세히 설명하는 풀 요청에 의견을 남겨야합니다.

따라서 필자 Waiting for Author는 저자가 솔루션에 대한 접근 방식을 망쳐 놓았다고 생각하지만 귀하의 의견을 염두에두면 코드를 사용할 수 있음을 의미합니다.

그리고 Rejected코드가 아무리 잘 작성되어 있더라도 이와 같은 변경을 지옥에서 받아 들일 수는 없습니다.

스스로에게 물어보아야 할 질문은 제안 된 목적에 대한 그룹의 아이디어가 Microsoft의 아이디어와 일치하는지 여부입니다.


1
이에 대해 : "저자는 문제를 해결 한 후 코드를 다시 검토해야 할시기를 알려 주어야합니다." -저자가 PR이 재검토 할 준비가되었음을 검토 자에게 경고하는 메커니즘이 있습니까?
Robert Sim

@RobertSim-검토 자 아래의 'Ping'옵션을 사용하여 재검토를 요청하는 메시지를 보냅니다.
killercowuk

이것을 보는 또 다른 방법 : Waiting for Author=이 기능을 원하지만 코드를 잘못 코딩했으며 제안한 방식으로 고칠 때까지 승인하지 않습니다. 그리고 Rejected=이 기능을 검토하고 싶지 않습니다.
goku_da_master
당사 사이트를 사용함과 동시에 당사의 쿠키 정책개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.