[jangh-lee]

유효하지 않은 링크에 # 을 추가하여 링크 완성 (anchor link)

[seokho-son]

@jangh-lee 컨트리뷰션 감사합니다!! :)

part of #681

[참고]
PR 내용이나 코멘트에 part of #681 ref #681 등과 같이,
관련된 이슈를 링크해주시면 더욱 좋습니다.. ^^

한 PR로 이슈 내용이 전부 해결된 경우,
fix #9999 와 같은 표시가 있으면
PR 머지와 함께 해당 이슈가 자동으로 close 되는 장점이 있습니다.

https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword

[seokho-son]

/lgtm

[seokho-son]

LGTM 은 “Looks Good To Me” 라는 뜻이고~
보통 리뷰어가 리뷰 후에 PR이 만족스러운 경우에 사용하는 오픈소스 약어.. 입니다.. ^^
So, LGTMLGTMLGTMLGTMLGTMLGTM ㅎㅎ

@jihoon-seo 님~ 확인 및 승인 부탁 드립니다!

[seokho-son]

PR의 제목, Commit message 등도 조금 더 명확하게 써주는 것이 좋습니다.. ^^
향후 컨트리뷰션을 위해서 습관을 들이시면 좋을 것 같습니다~~

[leedohun]

Merge를 한 다음에 제가 기여를 하면 될까요?? Merge를 하기 전에 기여를 하게 되면 충돌난다고 배웠어서 여쭤봅니다.

[seokho-son]

@leedohun
걱정하지 않으셔도 됩니다. ^^
오히려 충돌이 발생한다면, 이번 기회에 충돌이 발생하는 상황도 겪어보실 수 있으니 더 좋을 것 같습니다..!
(아마도 이번에는 발생하지 않겠지만..)

---

출동이 발생하는 상황과 해결 방법을 간략히 설명을 드리면 다음과 같습니다.

지금 상황과 같이
동일한 파일(e.g., file#1)을 수정하는 2개의 PR(커밋)이 오픈(e.g., PR#1, PR#2)되고,
PR#1이 먼저 머지가 되면, PR#1의 file#1이 main branch에 머지가 되니,

동일한 파일을 수정하는 오픈되어 있는 PR#2 에 영향을 줄 것입니다.

이때 GitHub에서 PR#2의 file#1과 업데이트된 main의 file#1 가 병합 될 수 있는지를 자동으로 비교해 줍니다.
그 결과, 만약 자동으로 머지가 불가능한 상황이면,

GitHub가 PR#2에 머지 컨플릭트가 발생했다고 메시지를 보여줍니다.

---

PR#2의 컨트리뷰터가 이 상황에 직면하면,

자신의 로컬 환경에서 main branch 의 최신 변경 사항을 읽어오고 (ex: git fetch upstream)
작업 브랜치에서 git rebase upstream/main 과 같은 형태로, 리베이스를 수행합니다. (리베이스... 는 베이스를 기준으로 커밋들을 병합하기 위한 방법으로 이해하시면 됩니다.)

이렇게 하면, rebase 를 하는 과정에서도 컨플릭트가 발생할 것입니다. (GitHub가 이미 알려줬듯이..)
이 컨플릭트는, 컨트리뷰터가 직접 살피면서 해결(resolve)해주셔야 합니다.

잘 처리가 되었다면, rebase가 완료될 것입니다.
rebase가 완료된 작업 브랜치를 다시 push 하시면, PR#2에 재반영되고
GitHub는 컨플릭트 여부를 다시 검증하여 오류 메시지를 제거해 줄 것입니다.

---

끝..^^

아마도 이번 사례에서는 컨플릭트가 발생하지 않을 것입니다.. Git가 자동으로 머지를.. 잘해주거든요.. ㅎㅎ
(동일한 파일이지만 수정하는 라인이 명확하게 나눠져있어서 아마도 자동으로 머지가 가능할 것입니다.)

[leedohun]

감사합니다!!

[jihoon-seo]

@jangh-lee @seokho-son @leedohun 감사합니다!
/approve

[seokho-son]

/approve

[seokho-son]

@all-contributors please add @jangh-lee for Documentation

[allcontributors]

@seokho-son

I've put up a pull request to add @jangh-lee! 🎉