TFS 코드 검토-주석에 대한 응답으로 업데이트 된 파일 표시


127

VS 2012 및 VS 2013 미리보기에 기본 제공되는 코드 검토 기능을 사용하기 시작했습니다. 검토 요청 및 의견 추가는 매우 간단 해 보입니다. 누군가 코드 변경을 요청하는 주석을 추가하면 요청자는 어떻게 변경하고 표시합니까?

따라서 프로세스는 다음과 같이 진행됩니다.

  1. 개인 1이 코드 검토를 요청합니다.
  2. 사람 2는 설명을 추가하고 "작업 필요"를 선택합니다.
  3. 사람 1은 필요한 변경을 수행합니다.

개인 1은 이제 개인 2에 대한 이러한 변경 사항을 어떻게 표시합니까? 주석을 추가하여 보낼 수 있지만 파일은 변경되지 않습니다. 원래 리뷰가 요청되었을 때 생성 된 변경 세트에서 파일이 있다고 가정합니다. 사람 1이이 검토를 종료하고 두 번째 검토를 요청해야합니까?

왜 변경했는지에 대한 대화 기록을 잃기 때문에 두 번째 검토를 작성하는 것이 최적으로 보이지 않습니다.

코드 검토의 기본 기능을 사용하는 방법을 보여주는 수많은 웹 사이트가 있지만 모범 사례를 보여주는 사이트가 있습니까? 예를 들어 누가 파일 옆의 확인란을 클릭해야합니까?


"모범 사례를 표시하는 사이트가 있습니까?"+1 ALM 레인저는 일반적으로 심층적 인 사용을위한 "goto guys"이지만, 내가 찾을 수있는 유일한 정보는 귀하의 질문에 답변하지 않습니다. Code Review를 사용하는 몇 가지 연습과 시나리오를보고 싶습니다.
DaveShaw

답변:


164

따라서 프로세스는 다음과 같이 진행됩니다.

  1. 개인 1이 코드 검토를 요청합니다.
  2. 사람 2는 설명을 추가하고 "작업 필요"를 선택합니다.
  3. 사람 1은 필요한 변경을 수행합니다.
  4. Person 1 코드 검토와 관련된 선반 세트를 업데이트합니다.
  5. 사람 1은 토론을 계속하기 위해 의견을 추가합니다
  6. 수락 될 때까지 2-5 단계를 반복하십시오.

검토와 관련된 선반 세트를 업데이트하는 데 필요한 단계는 다음과 같습니다.

  1. "코드 검토"창에서 "선반 세트보기"링크를 선택하십시오.
  2. "Shelveset Details"창에서 선반 세트 이름을 강조 표시하고 복사하십시오.
  3. "Pending Changes"창으로 이동하여 "Shelve"를 클릭하고 선반 세트 이름을 붙여 넣습니다.
  4. 선반 세트 교체 확인 대화 상자에서 예 버튼을 누릅니다.
  5. 이제 검토자가 업데이트 된 파일을 볼 수 있으며 검토 토론을 계속할 수 있습니다

내용을 명확히하는 데 도움이되는 스크린 샷을 포함 시켰습니다.


1) "코드 검토"창에서 다음과 같이 "선반 세트보기"링크를 선택하십시오.

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


2) "Shelveset Details"창에서 다음과 같이 선반 세트 이름을 강조 표시하고 복사하십시오.

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


3) "Pending Changes"창으로 이동하여 "Shelve"를 클릭하고 다음과 같이 선반 세트 이름을 붙여 넣습니다.

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


4) 선반 세트 교체 확인 대화 상자에서 예 단추를 누르십시오.

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


7
이 솔루션의 문제점은 모든 주석이 날아가는 것이 파괴적이라는 것입니다. 더 나은 해결책은 새로운 코드 검토를 작성하고 이전 코드를 관련 항목으로 설정하는 것입니다.
Bardia

15
선반 세트를 교체해도 주석 만 파일에 닿지 않습니다. Google 팀은이 방법을 성공적으로 사용하며 검토 의견을 풀지 않습니다.
Chad Dienhart

10
이것은 작동하지만 파일을 편집하고 선반 세트를 바꿀 때 행을 추가 / 제거하면 해당 파일에있는 주석이 잘못된 행을 가리 킵니다.
Chin

2
나는 보통 반복하고 싶지 않지만 위의 @Chin의 요점은이 대답을 완전히 불가능하게 만듭니다. 예, 선반 세트를 쉽게 업데이트 할 수 있지만 그렇게하면 본질적으로 이전의 모든 검토 주석이 잘못된 텍스트를 가리키게됩니다. (물론 리뷰 주석은 파일의 시작부터 절대 오프셋에 첨부되기 때문에.)
Michael Sorens

7
너무 수동적이고 이상적인 것은 아닙니다. TFS 또는 Visual Studio에는 연속 검토 간의 변경 사항을 비교할 내용이 있으므로 검토자는 마지막 검토 이후에 변경된 내용을 정확하게 볼 수 있습니다. 이와 같은 것이 없습니까? 아마도 플러그인이나 뭔가 ...
Iúri dos Anjos

6

본인은 올바른 절차가 사람 1을 변경하고 다른 검토를 요청하는 것이라고 생각합니다. 코드가 작동해야하는 경우 코드를 변경하므로 이전 버전을 다시 비교해보십시오. 주석을 살펴 보려는 경우 닫은 후에도 내역에 여전히 오래된 검토가 있습니다. 현재 직장에서 코드 검토 프로세스를 최적화하는 중입니다.


2

나는이 질문이 오래되었다는 것을 알고 있지만 다른 포스터가 지적한대로 여전히 지원되지 않습니다. chad가 제안한 해결책은 어떤 것에는 효과가 있지만 다른 것에는 이상한 행동을 보일 것입니다.

최근 TFS 팀은 아주 오래된 사용자 음성 요청 에 대한 솔루션 계획 단계를 시작하여 코드 검토 를 반복하여 코드 문제를 우아하게 해결해야하는 코드 검토업데이트 할 수 있었습니다.


1
큰. 나도 기대할 것이다. 나는 진정한 해결책을 환영합니다. 내 게시물은 단순히 문제를 해결하는 방법입니다.
차드 디엔 하트

바라건대 DevOps 풀 요청처럼 보이지 않을 것입니다. 스팸성 악몽입니다.
StingyJack

0

두 가지 다른 리뷰로이 작업을 수행해야합니다. 그러나 두 번째 검토로 역사를 유지할 수있는 방법도 있습니다. 당신이 필요한 것은 작업입니다.

이 워크 플로는 변경 집합 기반 검토에 대해 설명되어 있지만 선반 기반 검토에도 적용됩니다.

  1. 작업 만들기 1
  2. changeset1을 체크인하기 전에 task1을 관련 작업 항목으로 추가하십시오.
  3. 하나의 관련 작업 항목으로 변경 사항을 체크인하고이 변경 사항에 대한 검토를 요청하십시오.
  4. 작업 생성 2
  5. changeset2를 체크인하기 전에 두 작업을 모두 관련 작업 항목으로 추가하십시오.
  6. 관련된 두 개의 작업 항목으로 변경 사항을 체크인하고이 변경 사항에 대한 검토를 요청하십시오.

이제 두 번째 검토 요청에서 검토자는 관련 작업을 찾을 수 있으며 검토자가 작업 1을 찾는 경우 변경 세트 1과 검토 요청을 주석과 함께 볼 수 있습니다. 따라서 대화 기록을 잃지 않을 것입니다.

당사 사이트를 사용함과 동시에 당사의 쿠키 정책개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.