[#456] Swift 파일 헤더의 타깃명을 모듈 기준으로 정리한다#463
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Code Review
이번 PR은 DevLog 프로젝트 내 여러 모듈의 파일 헤더를 각 모듈명에 맞게 수정하고, 'ByID'를 'ById'로, 'PushNotificationSettings'를 'PushSettings'로 변경하는 등 네이밍 일관성을 개선하는 리팩토링을 수행합니다. 리뷰 피드백에서는 프로토콜 명칭 변경에 맞춰 구현체 파일 및 클래스 이름도 'FetchPushSettingsUseCaseImpl'로 일관되게 수정할 것을 제안했습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 연관된 이슈
🎯 의도
모듈 분리 이후 Swift 파일 상단 헤더 주석이 기존 앱 타깃명인
DevLog중심으로 남아 있어 현재 모듈 구조와 파일 정보가 일치하지 않는 문제 정리📝 작업 내용
📌 요약
+접미사 제거🔍 상세
DevLogDomain,DevLogData,DevLogInfra,DevLogPersistence,DevLogPresentation,DevLogWidgetCore,DevLogWidgetExtension기준 헤더 주석 수정FetchTodoByIdUseCase,FetchPushSettingsUseCase,TodoIdItem등 실제 타입명 기준 파일명 정리AppAssembler.swift,Tags.swift,ToolbarButtons.swift로 파일명 변경📸 영상 / 이미지 (Optional)