티스토리 뷰

https://github.com/developer0hye/Yolo_Label/pull/56

 

Visualize class names on the bounding box by e8035669 · Pull Request #56 · developer0hye/Yolo_Label

I also need class names on the bounding box, just like #43 says. So I try my best to made one, and mimic the "Darknet style". Hope you like it. 😊 By the way, I found QImage::scaled() returned an im...

github.com

 

YOLO Label 프로젝트에 유의미한 PR이 요청됐다.

 

기존에도 PR이 여러번 요청됐지만, 딱히 소스코드를 수정하고 merge 할 일이 없어서 그냥 바로 master 브랜치에 merge 했다.

 

근데, 이 PR을 머지하기전에 좀 수정한다음 master에 merge 시키고 싶었다.

 

 forked project로부터 나한테 날라온 PR에 커밋을 어떻게 하지...? 

 

찾아보니 설명이 너무 잘된 글이 있었다.

 

https://tighten.com/blog/adding-commits-to-a-pull-request/

 

Adding Commits to Someone Else's Pull Request

At Tighten, we do lots of open source work. We even have a whole day every week devoted to it. And, thankfully, we have many members of the open source community contributing pull requests to our projects and packages....

tighten.com

 

따라해보자

 

1. 내 project clone

git clone https://github.com/developer0hye/Yolo_Label.git

 

2. 수정할 PR 요청자의 forked project remote에 등록

 git remote add e8035669 https://github.com/e8035669/Yolo_Label

3. fetch 하기 이러면 PR 요청자가 떠간 forked project의 remote branch를 조회할 수 있게된다.

git fetch e8035669

4. local branch 만들고 remote branch 랑 연결시켜주기

git checkout -b visualize-class-name e8035669/master

5. 작업 후 커밋 & push

git commit -m "add visualize class names checkbox"
git push e8035669 HEAD:master

 

이렇게 하면 신기하게도 PR 이 날라온 forked project에 내 수정사항이 반영된다.

 

커밋을 추가하면 깃헙 PR 상태창에서 아래처럼 추가되는 것을 확인할 수 있다.

 

6. merge!

 

댓글
공지사항
최근에 올라온 글
최근에 달린 댓글
Total
Today
Yesterday
링크
«   2025/01   »
1 2 3 4
5 6 7 8 9 10 11
12 13 14 15 16 17 18
19 20 21 22 23 24 25
26 27 28 29 30 31
글 보관함