Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

지도 클래스 추상화 후 pr 올리겠습니다 #31

Closed
hayunyun opened this issue Dec 4, 2020 · 5 comments
Closed

지도 클래스 추상화 후 pr 올리겠습니다 #31

hayunyun opened this issue Dec 4, 2020 · 5 comments
Labels
question Further information is requested

Comments

@hayunyun
Copy link
Collaborator

hayunyun commented Dec 4, 2020

infowindow는 가능해졌는데, 코드를 그대로 옮기려니 페이지 코드들이 너무 복잡해져서요..
추상화를 하긴 해야할 것 같습니다 ㅠㅠ 오늘 해서 pr 올리겠습니다!
내용 확인은 제 브랜치에서 map 페이지 확인하시면 됩니다!

또 push하고 확인해보니, 커밋이 과거 커밋들까지 해서 무려 20개가 쌓여 있네요...;;
squash로 합치려고 했지만 변동사항을 확인하기가 어려우실 것 같아서 아직 하진 않았는데요.. 혹시 해결방법 아시나요? ㅜㅜ
이번엔 Github Desktop으로 해서 최대한 변동을 줄여봤는데 이게 웬 날벼락인지..

@hayunyun hayunyun added the question Further information is requested label Dec 4, 2020
@hayunyun hayunyun changed the title 클래스 추상화 후 pr 올리겠습니다 지도 클래스 추상화 후 pr 올리겠습니다 Dec 4, 2020
@SausageTaste
Copy link
Owner

Commit 개수는 아무 상관이 없고요. Files changes 리스트만 잘 관리하면 됩니다.

@SausageTaste
Copy link
Owner

commit 개수가 상관이 없는 이유는, 제가 이 repo에서 pr을 merge 할 때 squash를 강제하기 때문입니다. 어짜피 그 많은 commit 리스트는 squash 되어서 가려지니 상관없습니다.

@hayunyun
Copy link
Collaborator Author

hayunyun commented Dec 5, 2020

File changes 리스트를 보니 예전에 수정했던 것까지 남아있어서요..!

@SausageTaste
Copy link
Owner

main <- yoon1 이렇게 pr 만들어서 본 것 맞나요?
예전 수정 사항이 보인다는 게 뭔 뜻인지 모르겠습니다.
main branch로 update나 rebase 한번 해보세요.

@hayunyun
Copy link
Collaborator Author

hayunyun commented Dec 5, 2020

네 main <- yoon1 이렇게 본 것 맞습니다!
일단 pr에 예전 수정 사항을 구분해서 올렸습니다.
그런데 main과 conflit가 뜬 걸 보면 update가 안 되어 있던 것인가 싶습니다.
다음에도 똑같은 문제가 생긴다면 rebase하겠습니다!

@hayunyun hayunyun closed this as completed Dec 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants