Kim-aide / frontend

https://kim-aide.github.io/frontend/
0 stars 0 forks source link

[FEAT] 아이콘 텍스트 버튼 신규 생성 #19

Closed baesee0806 closed 2 months ago

baesee0806 commented 2 months ago

이슈 번호

Close: #2

작업 요약

논의

wzrabbit commented 2 months ago

아래의 사항을 확인해 주셨으면 좋겠습니다!

  1. PR의 작업 내역에 #9, #12 가 포함되어 있어요, 아마 배씨가 이 브랜치를 만든 이후 여러 PR이 머지된 상태라, 배씨의 브랜치의 위치가 최신 dev 브랜치를 기준으로 뒤에 있을 것이라는 생각이 들어요. 여러 다른 PR들은 적용되지 않은 상태로요. theme.ts를 예로 들었을 때, 실제로는 더 많은 색상이 추가되어 있는데, 배씨의 theme.ts에는 초기의 4가지 색상뿐이라, 머지되면 기존의 색상들을 추가했던 작업들이 손실될 거에요.

    • origin/dev를 기준으로, 배씨가 작업하신 커밋들만이 반영되도록 Rebase를 시도해 보실 수 있어요. 이 과정에서 충돌을 해결해 주시면 될 것입니다. 조금 더 풀어서 말씀드리면 지금까지의 모든 PR이 적용된 origin/dev를 시작점으로 배씨의 작업사항이 반영되도록 해 주자는 뜻이에요.
    • 제가 직접 할 수도 있지만, 배씨가 그걸 원하지는 않을 것이니, 배씨가 직접 시도해보시되 시행착오에도 불구하고 되지 않으면 저를 불러주세요. image
  2. PR명을 컨벤션에 맞게([FEAT] 내용) 변경을 부탁드려요, 아마 급하게 PR을 작성하시느라 그러셨던 것 같은데, 천천히 하셔도 됩니다 😅

  3. 이 PR은 현재 main 브랜치로 머지되도록 설정되어 있는데, dev가 되어야겠죠?

baesee0806 commented 2 months ago
  1. Rebase를 진행해서 충돌한거 수정 하면 될거 같습니다. 직접 하고 말씀 드릴게요
  2. 변경 바로 했습니다~~
baesee0806 commented 2 months ago

미친 제가 main으로 했었네요.... 와 확인해주셔서 감사합니다