Skip to content

Conversation

@hyunjuntyler
Copy link
Collaborator

@hyunjuntyler hyunjuntyler commented Dec 4, 2024

💡 요약 및 이슈

Swift format 룰을 수정하고 적용

📃 작업내용

  • swift format 룰 변경 및 적용
  • 추가 린트 걸린 부분 반영
// 변경된 pre-push
RESULT=$($FORMAT ./Projects --config .swiftformat)

// 추가된 룰
--disable trailingCommas

// 변경된 hooksPath
hooksPath = .githooks

적용 방법

  • 아래와 같이 로컬 설정에서 명시적으로 지정합니다.
git config --local core.hooksPath .githooks
  • 아래와 같이 명령어를 칠때
git config core.hooksPath

와 같이 잘 나와야 합니다.

빌드 테스트

  • 혹시몰라서 빌드 테스트도 완료했습니다.

린트 반영

  • 린트를 조금 반영해서 warning이 16개로 줄었습니다!!!! 🎉

🙋‍♂️ 리뷰노트

✅ PR 체크리스트

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

@hyunjuntyler hyunjuntyler self-assigned this Dec 4, 2024
@hyunjuntyler hyunjuntyler requested review from INYEKIM, Siwon-L and yongbeomkwak and removed request for Siwon-L and yongbeomkwak December 4, 2024 08:36
Copy link
Collaborator

@yongbeomkwak yongbeomkwak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

사실 저 커맨드를 쓴거면 .gitconfig 파일 안쓴거긴한데 ㅋㅋㅋ

@hyunjuntyler
Copy link
Collaborator Author

@yongbeomkwak 정확합니다.... 🥹 왜 안되는 것일까요

@hyunjuntyler hyunjuntyler force-pushed the task/fix_swift_format branch from 491c3b0 to 1c9aa94 Compare December 5, 2024 03:27
@hyunjuntyler hyunjuntyler merged commit 6672780 into develop Dec 5, 2024
2 checks passed
@hyunjuntyler hyunjuntyler deleted the task/fix_swift_format branch December 5, 2024 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants