Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Inner link error #683

Merged
merged 2 commits into from
Aug 20, 2021
Merged

Fix Inner link error #683

merged 2 commits into from
Aug 20, 2021

Conversation

leedohun
Copy link
Contributor

No description provided.

@leedohun
Copy link
Contributor Author

Inner link를 수정했습니다!! git PR를 자주 날려보니 이제 감이 조금씩 오는 것 같습니다~~ 감사합니다.

@jihoon-seo
Copy link
Member

@leedohun 감사합니다!
Conflict가 발생했네요 ㅎㅎ
해결하는 방법에는 크게 2가지가 있습니다.

  1. GitHub web UI에서 해결
  2. 이 commit을 찍은 개발 환경에서 해결

  1. GitHub web UI에서 해결

image

"Resolve conflicts" 누르시고

223~231 줄 (총 9줄) 에

<<<<<<< main
   - [개별 제어 시험](개별-제어-시험) (Namespace, MCIR, MCIS 등 개별 시험시, 오브젝트들의 의존성 고려 필수)
   - [통합 제어 시험](통합-제어-시험) (추천 테스트 방법) `src/testclient/scripts/sequentialFullTest/`
 - 3 단계: [멀티 클라우드 인프라 유스케이스 자동 배포](#멀티-클라우드-인프라-유스케이스)
=======
   - [개별 제어 시험](#개별-제어-시험) (Namespace, MCIR, MCIS 등 개별 시험시, 오브젝트들의 의존성 고려 필수)
   - [통합 제어 시험](#통합-제어-시험) (추천 테스트 방법) `src/testclient/scripts/sequentialFullTest/`
 - 3 단계: [멀티 클라우드 인프라 유스케이스 자동 배포](멀티-클라우드-인프라-유스케이스)
>>>>>>> main

이렇게 되어 있는 것을

   - [개별 제어 시험](#개별-제어-시험) (Namespace, MCIR, MCIS 등 개별 시험시, 오브젝트들의 의존성 고려 필수)
   - [통합 제어 시험](#통합-제어-시험) (추천 테스트 방법) `src/testclient/scripts/sequentialFullTest/`
 - 3 단계: [멀티 클라우드 인프라 유스케이스 자동 배포](#멀티-클라우드-인프라-유스케이스)

이렇게 3줄로 수정하시고

image

"Mark as resolved" 누르시면 됩니다.


  1. 이 commit을 찍은 개발 환경에서 해결

개발 환경에서
git remote -v 실행하셔서

origin  git@github.com:leedohun/cb-tumblebug.git (fetch)
origin  git@github.com:leedohun/cb-tumblebug.git (push)
upstream        git@github.com:cloud-barista/cb-tumblebug.git (fetch)
upstream        git@github.com:cloud-barista/cb-tumblebug.git (push)

이런 식으로
(즉, leedohun/cb-tumblebug repo는 origin으로,
cloud-barista/cb-tumblebug repo는 upstream으로 되어 있는지)
확인하시고

git fetch upstream main
git rebase upstream/main

실행하시면
conflict가 발생했다면서 resolve를 하라고 합니다.

그러면, VScode 나 vim 등을 이용하여 해당 파일을 열고
위의 1번에서와 같이 resolve를 하고 파일을 저장하고

git add <해당 파일>
git commit -m "새로운 커밋의 커밋 메시지"
git push origin main

하시면 됩니다. 😊

@seokho-son
Copy link
Member

컨플릭트가 발생했군요! 컨플릭트를 해결해보는 좋은 사례가 될 것같습니다.. 🥇
(FYI @hermitkim1 )

@seokho-son
Copy link
Member

@leedohun

차기 기여시 참고하시기 바랍니다~
참고: #682 (comment)

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

@yunkon-kim
Copy link
Member

@leedohun

오픈소스 협업을 위한 Git/GitHub 기본실습91페이지에 나오는 "오픈소스 프로젝트 개발 참여중 Rebase 가 필요한 상황" 부분이 앞으로 Conflict를 (최대한) 회피하시는데 도움이 될 것 같습니다. ^^

@seokho-son
Copy link
Member

@leedohun

오픈소스 협업을 위한 Git/GitHub 기본실습91페이지에 나오는 "오픈소스 프로젝트 개발 참여중 Rebase 가 필요한 상황" 부분이 앞으로 Conflict를 (최대한) 회피하시는데 도움이 될 것 같습니다. ^^

@hermitkim1 님 참고로~~

@leedohun 님께 제가 학습 차원에서 Conflict 상황을 미리 고려하지 않고 진행해보시길 제안 드렸습니다... ㅎㅎ

코멘트 참고: #682 (comment)

아마도.. 오픈소스 프로젝트에서는 Conflict 가 일어나는 것이 자주 발생할 수 있는 일반적인 상황이어서,
해결하는 방법을 알고 가시면 좋을 것 같습니다..^^

@yunkon-kim
Copy link
Member

넵! 참고차 공유 드려봤습니다 ^^

@leedohun
Copy link
Contributor Author

2번 방법은 오픈소스컨트리뷰톤에서 연습을 한 방법이라서 1번으로 해봤습니다.

@leedohun
Copy link
Contributor Author

2번 방법에서의 commit 메시지를 작성하는 부분은 없는지 궁금합니다.

@seokho-son
Copy link
Member

@leedohun 빠른 업데이트 감사합니다!! ^^

2번 방법에서의 commit 메시지를 작성하는 부분은 없는지 궁금합니다.

2번 방법에서는 commit 메시지를 아래 커맨드로 작성할 수 있습니다.

git add <해당 파일>
git commit -m "새로운 커밋의 커밋 메시지"
git push origin main

혹시 1번 방법에 대한 문의였다면~ (GitHub UI 사용)
저도 방법을 잘 모르겠어요. 커밋 메시지 지정하는 화면을 못 본 것 같습니다.
사실 저는 Conflict 상황에서 가능한 2번 방법 (메뉴얼?) 을
사용하고 있습니다.. ㅎㅎ;; 제 로컬환경에서 직접 작업하고 확인하는게 더 명확하고 안전한 것 같아서요~~
(소스코드의 경우 conflict 해결을 위한 추가 커밋을 하고 나서.. 빌드 테스트, 기능 테스트 등을 수행해야 겠지요..?^^)

@seokho-son
Copy link
Member

/lgtm

@github-actions github-actions bot added the lgtm This PR is acceptable by at least one reviewer label Aug 20, 2021
@seokho-son
Copy link
Member

/approve

@seokho-son seokho-son merged commit 6d8d7f3 into cloud-barista:main Aug 20, 2021
@seokho-son
Copy link
Member

@leedohun 컨트리뷰션 감사합니다!! :) 해당 PR은 승인 및 머지 완료하였습니다!

2개로 되어 있던 커밋 메시지는

main branch 에 머지 (승인) 할 때, squash merge 처리하였습니다.
squash merge는 PR의 커밋 메시지들을 1개의 커밋 메시지로 변경하여 머지한다는 뜻입니다.. ^^

커밋 히스토리 로그를 보시면~~
https://github.com/cloud-barista/cb-tumblebug/commits/main

아래와 같이, 1개의 커밋으로 머지된 것을 보실 수 있습니다.
image

그리고~~
1개의 PR에 의미 없는 커밋 메시지들이 여러게 포함되어 있으면,
오픈소스 관리자가 커밋 스쿼싱을 요청할 수도 있습니다. (커밋 로그 관리를 위해)
쿠버네티스 웹사이트 커밋 스쿼싱 참고

@seokho-son
Copy link
Member

@all-contributors please add @leedohun for Documentation

@allcontributors
Copy link
Contributor

@seokho-son

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

@jihoon-seo jihoon-seo linked an issue Aug 23, 2021 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR is acceptable by at least one reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invalid links in readme
4 participants