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

SwiftFormat 커밋에서 분리 #801

Open
baekteun opened this issue Jul 17, 2024 · 4 comments
Open

SwiftFormat 커밋에서 분리 #801

baekteun opened this issue Jul 17, 2024 · 4 comments
Labels
3️⃣ Priority: Low 우선순위 하

Comments

@baekteun
Copy link
Member

baekteun commented Jul 17, 2024

할 일

현재 Format을 한 번 돌리면 SwiftFormat이 Format을 한 커밋을 남기면서 자동 커밋이 되는데 git commit에 --amend 옵션이 있어요. 이거를 쓰면 push하기 전에 마지막 커밋에 Format을 한 내용물이 들어가지는데 이런 방향 어떤가요?

추가사항

Pros

  • 커밋 히스토리에 다량으로 분포해있는 이 "🎨 :: 코드 Formatting 적용" 커밋들이 남지 않게 돼요

Cons

  • Format을 할 때 새로운 커밋이 생기는게 아닌 push 마지막 코드의 커밋에 변경사항이 합쳐지니 변경사항의 자세한 트래킹이 어려울 수 있어요
  • 결국에는 기존 커밋에 변경사항을 추가하는거이기 때문에 나중에 커밋을 혹시나 분리하시는 작업을 한다면 의도하지 않았던 변경들도 같이 있을 수 있겠네요. 그나마 최근에는 굉장히 최소화되어서 Scope가 작업한 코드의 영역에만 해당되고 있는거같아서 큰 걱정거리까지는 아닌거같기도하네요
@baekteun
Copy link
Member Author

@KangTaeHoon
Copy link
Contributor

이렇게되면, "🎨 :: 코드 Formatting 적용" 의 커밋이 남지 않게되는건가요?

@baekteun
Copy link
Member Author

이렇게되면, "🎨 :: 코드 Formatting 적용" 의 커밋이 남지 않게되는건가요?

네네 마지막 커밋에 Format으로 인한 변경사항들이 들어가는 방향을 생각하고 있어요

@KangTaeHoon
Copy link
Contributor

ㅇㅎ, 포맷이 뭘 고쳐주나 나름 구경하는 맛도 있긴했는데, 이러나 저러나 저는 무관해요.

@KangTaeHoon KangTaeHoon added the 3️⃣ Priority: Low 우선순위 하 label Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3️⃣ Priority: Low 우선순위 하
Projects
None yet
Development

No branches or pull requests

2 participants