풀 리퀘스트는 작업이 main의 일부가 되기 전에 새로운 관점에서 다시 살펴볼 수 있는 기회입니다. 이 자습서에서는 고유한 변경 내용을 검토하고, 개선 사항을 적용하고, 웹 사이트의 첫 번째 기능을 병합합니다.
사전 요구 사항
- 회원님의
add-starred-list브랜치에 대한 열려 있는 풀 리퀘스트입니다. 아직 열지 않은 경우 코드 작성 및 저장을 참조하세요.
코드를 검토하는 이유
병합하기 전에 변경 내용을 검토하면 버그를 포착하고, 누락된 오류 처리를 발견하고, 작업이 최신인 동안 가독성을 향상시킬 수 있습니다. 혼자 작업하는 경우에도 신중한 검토는 소프트웨어 프로젝트를 정상 상태로 유지하고 나중에 공동 작업을 더 쉽게 만드는 습관을 쌓습니다.
끌어오기 요청에서 변경 내용 검토
GitHub의 풀 리퀘스트에서 변경된 파일 탭은 해당 브랜치가 추가하거나 수정한 모든 내용의 차이점을 보여줍니다. 신중하게 검토합니다.
- 내 풀 리퀘스트 보기를 클릭하여 GitHub의 내 풀 리퀘스트 받은 편지함으로 이동합니다. 또는 해당
stargazers-log리포지토리로 이동한 다음 Pull requests 탭을 클릭합니다. - 제목을 클릭하여
add-starred-list브랜치의 끌어오기 요청을 열 수 있습니다. - 변경된 파일 탭을 클릭하세요.
- 각 파일의 차이점을 읽고, 다음을 확인하세요:
script.js의 파일 이름 불일치와 같은 버그 또는 논리 오류events.json가 로드에 실패할 경우 어떻게 되는지와 같은 오류 처리가 누락되었습니다.- 누락된 텍스트 대안 또는 불분명한 레이블과 같은 접근성 문제입니다.
- 미래의 독자가 이해하기 어려울 수 있는 불분명한 부분
- 변경하려는 항목을 적어 두려면 한 줄 위로 마우스를 가져가고 메모를 입력한 다음 메모를 클릭합니다.
선택 사항: AI 두 번째 의견 가져오기
편집기에서 동일한 변경 내용을 검토하고 두 번째 관점을 제공하도록 요청할 공동 파일럿 채팅 수도 있습니다.
-
편집기에서 공동 파일럿 채팅을 엽니다.
-
다음과 같은 프롬프트를 사용하여 변경 내용을 검토하도록 요청합니다.
Text Review my changes for bugs, missing error handling, and accessibility issues.
Review my changes for bugs, missing error handling, and accessibility issues. -
제안을 읽고 기능을 개선하는 제안을 결정합니다. 변경 내용을 항상 제어할 수 있습니다.
팁
자신의 코드를 검토할 때 Copilot를 유용하게 사용하셨다면, 유료 플랜에 가입하여 추가 AI 크레딧과 Copilot 코드 검토를 이용할 수 있으며, Copilot를 검토자로 추가하면 풀 리퀘스트를 자동으로 검토할 수 있습니다.
자세한 내용은 설명서의 GitHub Copilot을 직접 설정하기 및 GitHub Copilot 코드 검토 정보.
피드백 적용
검토 노트 및 동의한 제안을 공동 파일럿 채팅 업데이트로 전환합니다.
- 편집기에서 영향을 받는 파일을 업데이트하여 찾은 내용을 해결합니다.
- 에서 GitHub Desktop커밋 메시지(예:
Address review feedback커밋)를 입력한 다음 커밋 # 파일을 클릭하여 별표 목록을 추가합니다. - 원본 푸시를 클릭하여 끌어오기 요청을 새 커밋으로 업데이트합니다.
끌어오기 요청 병합
변경 내용에 만족하면 해당 변경 내용을 main에 병합하세요.
- GitHub에서 풀 리퀘스트로 돌아가서 페이지를 새로 고침합니다.
- 대화 탭을 클릭합니다.
- 병합 끌어오기 요청을 클릭한 다음 병합 확인을 클릭합니다.
- 브랜치 삭제를 클릭하여
add-starred-list브랜치를 정리하세요.
진행률 확인
완료된 작업을 반영하도록 프로젝트 보드를 업데이트합니다.
- GitHub리포지토리의
Stargazers log프로젝트 탭에서 프로젝트 보드로 이동합니다. Display a list of starred repositories중인 **** 항목에서 완료로 항목을 끌어옵니다.
빌드할 기능이 더 있는 경우 새 문제를 만들고 다음 주기를 시작할 때 진행 중 으로 이동합니다.
전체 워크플로
이제 전체 개발자 워크플로를 실행했습니다.
- 문제 및 프로젝트 보드를 사용하여 작업을 계획했습니다.
- 브랜치를 만들고 기능을 구현했습니다.
- 끌어오기 요청을 열었습니다.
- 자신의 변경 내용을 검토했습니다(선택적으로 Copilot의 도움을 받아).
- 기능을
main에 병합했습니다. - 진행 상황을 반영하도록 프로젝트 보드를 업데이트했습니다.
달성한 작업
| 과업 | 결과 |
|---|---|
| 변경된 파일 탭에서 변경 내용 검토 | 끌어오기 요청의 파일 변경 탭을 열고 diff를 읽었습니다. |
| 피드백을 적용하고 선택적 AI 지원을 받았습니다. | 변경 내용을 적용하고 검토를 요청했을 공동 파일럿 채팅 수 있습니다. |
| 회원님의 기능을 병합했습니다 | 변경 내용을 main에 병합했습니다. |
| 진행률을 업데이트했습니다. | 프로젝트 보드에서 문제를 완료 로 이동했습니다. |
다음 단계
- 웹 사이트를 자동으로 업데이트하는 라이브 URL에 소프트웨어 프로젝트를 게시합니다. AUTOTITLE을 계속 진행합니다.