[Modney]

Add comment for exported type SpiderAllListWrapper.

To fix task-2 of #783.

[seokho-son]

@Modney 컨트리뷰션 감사합니다!! : )

향후에 오픈소스에 기여하실 때 보완해주시면 좋은 사항을 공유 드릴게요 ^^

• 커밋 메시지는 log 상으로 확인했을 때 의미가 명확하면 좋습니다. (명확할수록 좋아요 ^^)
  • 예를 들면, Fixed Commit for Task2 보다는 Add comment for exported type SpiderAllListWrapper 이런 형태가 더 좋습니다.. ^^
• 마찬가지로 PR 제목도 의미가 명확하면 좋습니다. (PR은 제안서 라고 생각하시면 됩니다.)
  • PR 예시 (Display output of MCIS status script in a table #738)
  • 이번 PR의 제목은 다음과 같이 수정해보겠습니다.. ^^
    (Add comment for exported type SpiderAllListWrapper)
• PR이 어떤 제안인지 내용으로 알려주시면 좋습니다. 특히 더 엄격하게 관리하는 오픈소스 프로젝트에 기여하실 때는 최대한 상세히 알려주시는 것이 좋아요 ^^
  • 예시로 PR 내용도 수정해두겠습니다. 향후 기여하실 때 참고하시기 바랍니다.. ^^

---

• 커밋은 의미가 명확한 커밋들로만 구성하면 좋습니다. 현재는 커밋이 2개로 구성이 되어 있습니다.
  • Fixed Commit for Task2
  • Merge branch 'cloud-barista:main' into main
• 이 두개의 커밋 중에서 Merge branch 'cloud-barista:main' into main 는 업스트림 저장소 커밋 히스토리에 의미가 없는 커밋입니다.. ^^ 이런 상황이 발생하면, 보통은 메인테이너가 커밋 스쿼싱 (커밋들을 1개로 수정하여 PR을 업데이트하는 작업)을 요청 드릴 수 있습니다. 이번에는 제가 처리해두겠습니다~~ 향후에 조금 더 익숙해지시면 커밋 스쿼싱도 해보도록 할게요! :)

[seokho-son]

feature-01 로컬 브랜치에서 커밋을 1개만 했는데, 현재 PR에 왜 커밋이 2개가 포함되었는지 궁금하실 것 같아요.

• 커밋1번: Fixed Commit for Task2
• 커밋2번: Merge branch 'cloud-barista:main' into main

---

작업하신 내용을 보면,
feature-01 로컬 브랜치에서 수정하신 다음에 @DalboDosa 라는 Github 사용자명으로 커밋을 수행하신 것 같아요.

커밋 1번: 6e541d6

그런데, feature-01 로컬 브랜치를 다른 계정인 @Modney 의 cb-tumblebug 오리진 저장소에 푸시하신 것 같네요. (향후에는 커밋한 사람도 동일하게 해주시면 좋아요 ^^)

---

그리고

업스트림 저장소(cloud-barista/cb-tumblebug)의 main 브랜치로 PR을 바로 생성하신 것이 아니라,

오리진 저장소(Modney/cb-tumblebug)에 푸시한 feature-01를 Modney/cb-tumblebug main 브랜치에 merge를 먼저 하신 것 같아요. 이렇게 merge를 하시면, 기존의 커밋이외에 merge 커밋이 생기게 됩니다.

커밋 2번: f80259b

이때 @Modney 님의 오리진 저장소에는 커밋이 2개 반영이 되었을 것입니다.

---

마지막으로 업스트림 저장소(cloud-barista/cb-tumblebug)에서 PR을 오픈하셨고,
오리진 저장소(Modney/cb-tumblebug)의 main 브랜치를
-->
업스트림 저장소(cloud-barista/cb-tumblebug)의 main 브랜치에 merge해달라고 요청하신 것입니다.

오리진 main 브랜치에 2개의 커밋이 있었으니, 해당 PR에서도 2개의 커밋이 포함되어 있는 것입니다.

---

지금은 개념이 좀 어렵게 느껴지실 수도 있을 것 같아요 ^^
의미를 파악하고 기여를 계속하시다 보면 차차 익숙해지실 것입니다~~

[seokho-son]

기여자마다 리모트 저장소 활용 방법이 다르긴 합니다.. ^^
@hermitkim1 께서 첨언해주실 사항이 있으실까요~?

[yunkon-kim]

@Modney

먼저, 노력하시며 발전하는 모습에 박수를 보내드리고 싶습니다 👏
지금 혼란스러우신 몇가지를 이해하고 나면 앞으로 걱정 없으실 것 같아요~

관련하여 차기 세션에 관련 내용을 추가하였습니다. 다음 세션에서 충분히 이해하실 거라 믿습니다.
참고: CB-Larva 및 Contribution에 대해 조금 더 자세히 들여다보기 세션

추가로, 자신만의 기여 루틴(?)을 만들어보시는 것도 도움이 되실 것 같습니다.
저번에 기여 순서 공유 드렸던 내용 참고하셔서 루틴을 만들어보세요 ^^

[seokho-son]

/lgtm

[seokho-son]

/approve

[seokho-son]

@all-contributors please add @Modney code

[allcontributors]

@seokho-son

I've put up a pull request to add @Modney! 🎉