Skip to content

[#461] 빌드, 테스트 타깃에 SwiftLint를 구성한다#462

Merged
opficdev merged 8 commits into
developfrom
fix/#461-swiftlint
May 16, 2026
Merged

[#461] 빌드, 테스트 타깃에 SwiftLint를 구성한다#462
opficdev merged 8 commits into
developfrom
fix/#461-swiftlint

Conversation

@opficdev
Copy link
Copy Markdown
Owner

@opficdev opficdev commented May 16, 2026

🔗 연관된 이슈

🎯 의도

  • 모듈러 아키텍처 전환 이후 각 모듈 스킴에서 SwiftLint가 누락되었으므로 설정 복구

📝 작업 내용

📌 요약

  • 각 모듈 Sources/Tests 타겟에 SwiftLint BuildToolPlugin 적용
  • 모듈별 .swiftlint.yml 추가 및 루트 SwiftLint 설정 참조 구조 구성
  • 테스트 전용 SwiftLint 설정 분리
  • SwiftLint 적용 후 발생한 일부 린트 위반 정리

🔍 상세

  • DevLogApp, DevLogCore, DevLogData, DevLogDomain, DevLogInfra, DevLogPersistence, DevLogPresentation, DevLogWidgetCore에 SwiftLint 플러그인 연결
  • Sources 설정은 루트 .swiftlint.ymlparent_config로 참조하도록 구성
  • Tests 설정은 .swiftlint-tests.ymlparent_config로 참조하도록 구성
  • 테스트 코드의 한글 함수명을 유지하기 위해 테스트 전용 설정에서 identifier_name 비활성화
  • WidgetSnapshotUpdater의 히트맵 스냅샷 갱신 API에서 테스트 전용 성격의 selectedActivityKinds 오버로드 제거
  • 히트맵 activity 선택값은 실제 저장소 설정을 통해 읽도록 정리
  • 테스트 fixture 튜플을 Fixture 타입으로 분리해 large_tuple 위반 해결
  • 불필요한 AppleAuthResponse 생성자 제거

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this May 16, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

이번 pull request는 SwiftLint를 빌드 도구 플러그인으로 통합하고 모듈별 린트 설정을 표준화했습니다. 또한 WidgetSnapshotUpdater의 인터페이스를 단순화하여 내부에서 설정을 직접 참조하도록 수정하였으며, 테스트 코드에 Fixture 패턴을 도입하여 구조를 개선했습니다. 리뷰 과정에서 WidgetSnapshotUpdaterSpy의 접근 제한자를 private으로 유지하여 캡슐화를 강화하라는 의견이 제시되었습니다.

Comment thread Widget/DevLogWidgetCore/Tests/Sync/WidgetSyncEventHandlerTests.swift Outdated
@opficdev opficdev changed the title Fix/#461 swiftlint [#461] 빌드, 테스트 타깃에 SwiftLint를 구성한다 May 16, 2026
@opficdev opficdev merged commit cdda8d9 into develop May 16, 2026
1 check passed
@opficdev opficdev deleted the fix/#461-swiftlint branch May 16, 2026 03:59
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.

빌드, 테스트 타깃에 SwiftLint를 구성한다

1 participant