우리 팀은 소스 제어를 위해 Team Foundation Server를 사용하고 있으며 오늘 체크인하기 전에 몇 가지 버그와 연기 테스트 응용 프로그램을 수정했지만 코드를 주석 처리하는 것을 잊었습니다. (이 코드는 UI를 조금 이상하게 만들었습니다.)
코드를 체크인하기 전에 어떤 모범 사례가 있는지 알고 싶습니다. 다시 이런 종류의 실수를하고 싶지 않습니다.
답변
내가하는 습관에서 얻은 한 가지는 항상 체크인하기 전에 체크인하려는 모든 파일의 차이점을 보는 것입니다.
답변
주석 처리 된 코드를 체크인해서는 안됩니다. 체크인 전에 주석 처리해야하는 코드가있는 경우 잘못된 코드입니다.
규칙에 관해서는 :
- 최신 정보 받기
- 병합 충돌 수정
-
짓다
3.1 빌드 오류 수정
-
테스트 실행
4.1 고장난 테스트 수정
-
1로 가십시오 (얻을 새로운 것이 없을 때까지)
모든 단계가 완료된 경우에만 체크인하십시오.
다른 모범 사례 :
- 체크인 할 파일 목록을 검토하여 예상 파일인지 확인하십시오.
- 각 파일의 변경 사항 검토 (삭제 / 추가 / 차이)
답변
내가 너무 여기에 pantsweasel의 되려고 노력하고 있지 않다,하지만이 질문에 가정 (모든하지만 대답 중 하나) 대부분 같은 TFS, SVN, 억지로, 등 중앙 집중식 VCS에 적용
페어 충분히, 그것은 무엇을의 OP가 사용 중입니다.
그러나 DVCS (예 : Mercurial 및 Git)를 사용하는 경우 일반적으로 체크인 대기하지 않아야하며, diff, 최신 정보, 병합 등과 같은 답변에 언급 된 대부분의 항목은 필요하지 않습니다. . 코드 검토 및 테스트와 같은 것조차 체크인 후 수행하는 것이 좋습니다 (아마도 푸시하기 전에 …)
지금까지 본 예외는 작업 항목과 관련이 있습니다. 물론 체크인에 대한 의견도 좋습니다 …
답변
다른 답변에서 보지 못한 세 가지 :
새 파일 포함
- 변경 목록에 포함되지 않은 새 파일을 찾으십시오.
- Perforce와 같은 SCM에만 해당 될 수 있습니다. 변경 사항에 대한 모든 정보를 제공해야합니다.
변경되지 않은 파일 되돌리기
- 나는 다른 사람들의 변경 사항을 볼 때 싫어하고 9 개의 파일이있는 변경 목록이 있지만 그중 3 개만 수정되었습니다.
- 또한 공백이나 의미없는 변경 사항이있는 파일을 되돌립니다.
제출 한 커밋 확인
- 커밋 후에 빌드가 녹색으로 유지되는지 확인하십시오.
- 커밋 후에 동기화, 빌드 및 실행하는 두 번째 컴퓨터를 사용했기 때문에 문제가 발생하면 쉽게 뛰어들 수 있습니다.
Git을 사용할 때 두 가지 :
원자 커밋 :
- 커밋을 위해 개별 기능 변경 만 준비하십시오.
- 커밋을 가능한 작게 만드십시오. 패치, 되돌리기 및 이해하기 쉽도록하십시오.
- 내가 사용
git add --patch필요한 경우 여러 부분으로 내 변화를 분할 할 수 있습니다.
요약하는 동안 차이점보기
git commit --verbose커밋 메시지를 입력하는 동안 항상 변경 사항을 확인할 수 있도록 항상 체크인 합니다. (또는 패치 된 git-vim 을 사용하여 diff를 표시합니다.)- 이를 통해 변경 사항을 쉽게 처리하고 커밋 전체를 설명 할 수 있습니다. 때때로, 나는이 단계에서 의도하지 않은 변화를 잡습니다. (변경 내용을 설명하면 변경에 대해 생각하는 데 도움이됩니다.)
답변
팀 서버에 적용하는 ‘좋은 습관’몇 가지 항목은 매우 간단합니다. 먼저, 체크인하기 전에 항상 코드를 충돌시킬 수있는 다른 사람을 확인하지 않도록 항상 최신 상태를 유지하고 로컬 빌드를 실행해야합니다. 또한 서버가 아닌 로컬 컴퓨터의 코드 충돌을 처리하십시오. 최신 코드가 다운로드 된 코드가 빌드되고 제대로 작동하는 것으로 확인되면 다음 단계를 수행 할 수 있습니다. 자동화 된 테스트를 실행 한 다음 체크인을 시작하여 여전히 제대로 작동하는지 확인하십시오. 그런 다음 확실하게 최신 정보를 다시 받으십시오.
TFS 관리자는 모든 체크인에 대해 주석을 달 수 있습니다. 작업 여부에 관계없이 항상 귀하의 작업에 대한 체크인 의견을 작성하는 것이 좋습니다. 그렇게 할 수있는 옵션이 있다면 시행하십시오. 주석은 최소한 코드를 마지막으로 체크인 한 이후 변경 한 내용에 대한 일반적인 요약이어야합니다. 이렇게하면 문제가 발생하면 체크인을 살펴보고 대략적인 내용을 확인할 수 있습니다. 체크인시 변경되었습니다. 깨진 빌드를 훨씬 쉽게 디버깅 할 수 있습니다.
또한 TFS 관리자 권한이있는 경우 체크인시 롤링 빌드를 시행하고 (체크인이 문제를 겪는 경우 다른 사람이 즉시 알 수 있도록) 게이트 된 체크인을 수행하도록 서버를 설정할 수 있습니다 ( 체크인 된 코드가 빌드를 중단하면 서버가 거부합니다) 또는 단순히 버그를 생성하여 빌드를 위반 한 사람에게 할당하도록 할 수 있습니다.
모든 것을 제대로 유지하기 위해 켜거나 끌 수있는 몇 가지 다른 옵션이 있으며, TFS 관리자에게 설정하여 물건을 깨끗하고 깨끗하게 유지하도록 제안 할 수 있습니다.
답변
저주 단어 찾기 및 바꾸기 😉
답변
Windows에서 체크인하는 경우 코드에 보이지 않는 ^ M 문자가 없는지 확인하십시오 .UNIX의 편집기는 종종 오류 / 경고를 유발합니다.
또한 탭을 시도하고 교체하십시오. 다른 사용자는 4 개의 공백이있는 탭 스탑을 다르게 보게 될 것이며 8은 코드 가독성에 좋지 않습니다.
최선의 방법 IMHO는 미리 정의 된 스크립트로 조직의 코딩 지침에 따라 코드를 확인하도록하는 것입니다. 많은 소스 제어 시스템에이 기능이 있습니다.