code-review

-1열

1답변

TFS에 대한 병합 지점 승인을 시행합니다.

저는 SQL 개발 환경에서 일하고 있습니다. DEV, TEST 및 LIVE의 세 가지 분기가 있습니다. 개발자가 뭔가를하고 Test System에 배포하려고 할 때마다 코드 검토를 수행하도록 TFS 내의 메커니즘이 필요합니다. 코드 검토 체크인 정책을 적용하면이 작업을 수행 할 수 있습니다. 하지만 모든 체크 인에서 코드 검토를 실행하고 싶지는 않지만 누군

tfs merge code-review release-management 2017-01-09

---

17열

2답변

더 커밋이 있는지 끌어 오기 요청이 승인되면 새로운 자동

을 저지른 후 github의 검토를 해제 승인하는 방법 : 끌어 오기 요청이 자동으로 않은 승인 상태로 가야한다. 이 작업을 수행 할 수 있습니까?

github code-review github-codereviews 2017-01-24

---

0열

3답변

요청에 변경 요청을 포함시키는 올바른 방법

요청 된 변경 사항을 PR에 포함시키는 올바른 방법은 무엇입니까? 리뷰어가 내 PR에서 일부 변경을 요청했다고 가정합니다. 메서드 이름 바꾸기, 종속성 업데이트 실행 취소 등 변경하고 새로운 커밋을 수행합니까? 또는해야합니까? change past commit to include the requested changes? or reset and change

git github code-review pull-request 2017-01-26

---

0열

1답변

Git review-pack .gitconfig에 리뷰어 그룹을 추가하는 방법

gitconfig에 개별 리뷰를 추가하는 방법이 있지만 비슷한 방법으로 리뷰어 그룹을 추가하고 싶습니다. 어떤 방법이 있습니까? 참조 : https://stackoverflow.com/a/24582974/1084174

git code-review gerrit git-config 2017-01-27

---

-1열

2답변

GitHub pull 요청을 코드 검토에 효과적으로 만들기

우리는 GitHub- master와 dev에 2 개의 브랜치를 가지고 있습니다. 모든 개발자가 Dev에서 변경 한 다음 끌어 오기 요청을 올리고 코드 검토를 마친 후이를 마스터에 병합합니다. 이 접근법의 문제점은 모든 개발자가 동일한 분기로 푸시되므로 변경 사항을 고유하게 식별하기가 매우 어려워집니다. 관련없는 코드조차도 풀 요청 (다른 개발자가 수행)의

git github code-review branching-and-merging branching-strategy 2017-01-27

---

-1열

1답변

코드 리뷰 점수를 재설정하지 않고 Gerrit에서 패치 세트를 푸시 할 수 있습니까?

때로는 코드 검토 과정을 거치지 않고 오타를 수정하거나 메모를 수정하는 등의 코드를 외관상으로 변경하려고합니다. 코드 검토 점수를 유지하면서 새 패치 세트를 푸시하라는 명령이 있습니까? 감사합니다.

git code-review gerrit 2017-01-27

---

0열

1답변

첫 번째 새로운 Gerrit 대시 보드 만들기

대시 보드가 정의되지 않은 Gerrit 프로젝트가 있습니다. 동일한 Gerrit 서버에서 여러 프로젝트를 쉽게 구분할 수 있도록 프로젝트 중 하나에 대해 새 대시 보드를 만들고 싶습니다. official documentation (적어도 v2.13.5-2456 현재) 대시 보드가 생성 될 지점이 이미 존재한다고 가정합니다. 이는 설치에 해당하지 않습니다.

code-review gerrit dashboard 2017-02-01

---

1열

1답변

검색 클래스입니다. 패턴 조언 요청

친구들. 현재 복잡한 검색을하고 있습니다. 나는 다음 학급을 가지고있다 : App\Search\Search App\Search\OrderBy\Relevance App\Search\OrderBy\Priority App\Search\OrderBy\WordMatch App \ Search \ Search는 주요 검색 클래스이다. \ OrderBy의 모든

laravel design-patterns code-review 2017-02-02

---

0열

1답변

마스터 브랜치의 코드 검토 프로세스에 대한 도전

QA 팀의 권장 사항으로, 프로젝트에서 단일 마스터 브랜치를 유지 관리하고 있으며 모든 팀 구성원이 브랜치를 직접 마스터 링하고 있습니다. 이렇게하면 CI가 마스터에서 실행되기 때문에 CI가 간단하며 CI에 첨부 된 모든 UT, 정상 도구 등이 있으며 모든 것이 제자리에 있습니다. 그러나 문제는 코드 리뷰에 있습니다. 모두가 수시로 마스터하기 때문에 팀원은

git continuous-integration code-review 2017-02-03

---

0열

1답변

비즈니스 로직 내에서 모델 속성을 여러 번 확인하십시오.

비즈니스 로직 내에서 모델 속성을 여러 번 확인해야하는 경우가 있습니다. eg: if(this.model.attributes.AcmeData.attributes.isEmpty){ } 이 검사는 여러 번 발생합니다. 이 객체를 여러 번 확인해야 또는 우리가처럼, 여러 위치에 사용되는 지역 변수에 저장해야합니다 : var acmeDataFlag =

javascript code-review 2017-02-16

---