Tails 구성 요소를 검토하는 방법. 👨‍💼

Tails가 처음 대중에게 공개되었을 때. DevDojo는 기여를 위해 저장소를 열었습니다. 풀 리퀘스트 후 풀 리퀘스트가 들어오도록 합니다. 일부는 첫 번째 요청에서 프로젝트에 병합할 준비가 되었습니다. 그러나 일부는 약간의 변경이 필요했습니다. 다음은 GitHub CLI 도구 및 Live Server 의 도움으로 이러한 풀 리퀘스트를 검토하는 방법입니다.

요청입니다.

요청이 저장소에 들어오면 가장 먼저 웹에서 요청을 봅니다. 나는 기여자가 작성한 요청 본문을 살펴봅니다. 구성 요소의 이미지를 추가한 경우. 그런 다음 브라우저에서 코드를 빠르게 살펴보고 코드를 체크아웃하기 전에 눈에 띄는 것이 있는지 확인합니다.

체크아웃.

요청과 코드에 대한 간략한 개요를 살펴보고 검토를 시작하기로 결정했습니다. 나는 열고vscode 꼬리의 나의 개인 포크로 여행한다. 거기에서 다음 명령을 실행합니다.

gh pr checkout --repo thedevdojo/tails {pr}

이 경우 {pr}는 pull 요청 ID 번호입니다. 이전에 GH로 결제를 시도한 적이 있다면. 처음 실행할 때pr checkout 리모컨이 여러 개 있다는 것을 알 수 있습니다. 제 경우에는 두 가지가 있습니다. 내 포크는 원점과 업스트림에 있습니다.
GH가 기본으로 사용할 저장소를 묻는 프롬프트가 나타납니다. 내 원본을 내 원본으로 설정했기 때문에 검색할 저장소를 지정하기 위해 --repo 플래그를 전달해야 합니다.

이 명령이 실행된 후 GH는 요청에서 분기를 풀다운하고 내 로컬 버전의 Tails에 새 분기를 추가하고 해당 분기를 체크아웃합니다.

리뷰.

여기서부터 재미가 시작됩니다. 내가 모든 것을 로컬에 가지고 있을 때. 구성 요소를 열고 내가 설치한 린터가 파일을 빠르게 스캔하도록 합니다. 그럼 내가 게으른 이후. landmark 템플릿을 열고 헤드를 복사합니다. 구성 요소에 붙여넣고 Live Server에서 파일을 엽니다. 거기에서 구성 요소가 다른 브라우저에서 어떻게 보이는지 볼 수 있습니다. 구성 요소의 응답성을 확인하고 제안하고 싶은 변경 사항을 가지고 놀아보세요.

로컬 브랜치에 요청이 있으면 구성 요소를 테스트할 때 더 많은 자유를 얻을 수 있습니다. 또한 필요에 따라 초기 요청에 내 자신의 pull 요청을 다시 제출하기 위해 요청의 새 분기를 쉽게 만들 수 있습니다.

제안.

이제 검사를 완료하고 코드를 살펴보았으므로 제안할 준비가 되었습니다. pr view 명령을 사용하면 브라우저에서 요청을 쉽게 열 수 있습니다.

gh pr view --repo thedevdojo/tails --web {pr}

브라우저에서 요청을 열면 내 제안을 공식화할 수 있습니다. 나는 보통 메인 리뷰 댓글에 내 제안을 적는 것으로 시작합니다. 내가 작성한 것이 인라인 주석으로 더 적합하다고 생각되면 전체 주석에서 해당 부분을 제거하고 인라인 주석으로 이동합니다. 이를 통해 코드 변경에 대한 직접적인 제안을 할 수도 있습니다.

내 모든 제안을 작성한 후. 내가 제안한 모든 것을 마지막으로 한 번 살펴봅니다. 마지막 간과에서 내가 주로 찾는 것은입니다. 기여자가 이해할 수 있는 방식으로 내 제안을 설명했습니까? 그렇지 않은 경우 이해할 수 있을 때까지 다시 작성하고 변경합니다.

마지막으로 Request changes 상자를 선택하고 내 리뷰를 제출할 시간입니다. 그 후 제출합니다. 다음 요청으로 넘어가 첫 번째 요청에서 응답이나 변경 사항을 기다립니다. 그런 다음 요청이 승인되고 병합될 때까지 헹구고 반복합니다.

결론적으로.

리뷰 제출은 몇 단계로 요약하면 그렇게 복잡하지 않습니다. 나 자신을 위한 배움의 경험이기도 하다. 검토 중인 코드에 대해 더 신중하게 생각해야 하기 때문입니다. 그리고 기여자들이 코드를 구조화하고 문제를 해결한 방법을 확인하십시오.

결국. 그것은 단순한 전제로 귀결됩니다. 코드 검토는 재미있습니다.