Repository navigation
maintenance(konsist): MVI 가드 예외 목록과 규약 문서의 낡은 항목·개수·담당 문구를 정리한다 (#2246) - #2256
Merged
OnlyReadOneBook merged 1 commit intoOct 5, 2026
Merged
Conversation
Contributor
|
열린 PR 중 이 PR 과 같은 파일을 서로 다르게 고친 것이 있다. 어느 쪽이 먼저 머지되든 나중 쪽은 base 를 병합해야 통과한다 (#1750).
머지 자체는 GitHub 이 막는다 — 한쪽이 들어가면 다른 쪽은 CONFLICTING 이 되어 merge queue 에 못 들어간다. 이 코멘트는 리컨사일러가 매 실행 현재 상태로 고쳐 쓴다. 해소되면 |
Sadturtleman
approved these changes
Oct 5, 2026
OnlyReadOneBook
deleted the
maintenance/2246-mvi-guard-allowlist-cleanup
branch
October 5, 2026 07:07
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.
📌𝘐𝘴𝘴𝘶𝘦𝘴
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
PENDING_MVI_MIGRATION에서 develop 에 없는feature.receiver.presentation.home.ReceiverHomeViewModel과 철회 스택용 주석 2줄을 지웠다. 실제 클래스는 refactor(home): 수신자 홈 재진입 갱신 이관과 home 정리 3건 (#1452) #1666 으로feature/home에 옮겨져ISSUE_1808_HOME에 이미 있다. 매 실행 나던 해소 경고 1건이 없어진다.ISSUE_1803_RECEIVER에서UNASSIGNED_RECEIVER로 바꿨다. 철회된 구 수신자 흐름의 현존 예외 7개는 그대로다.PENDING_ARRIVAL_ONBOARDING에서UNASSIGNED_ONBOARDING으로 바꿨다. 해소 경고 문구의PENDING_ARRIVAL_*안내도 같은 이유로 바꿨다.docs/convention/mvi.md)을 규칙 B 범위 밖이라는 사실로 고쳤다.MainViewModel판정은 refactor(platform): 최종 앱 루트 책임 판정과 MVI 전환 종료 검사 (#1809 분리) #2022 에 남는다../gradlew :konsist:test --rerun-tasks통과(56건, 실패 0). MVI 가드 해소 경고는 develop 원본에서 1건, 이 브랜치에서 0건이다.:konsist:ktlintCheck통과.🧪 CI Test Plan
{"androidTest":{"mode":"none","reason":"konsist 테스트 주석·예외 목록과 docs 만 변경, 프로덕션·계측 경로 변경 없음","tests":[]}}📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
없음
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴
mvi.md182행 주변을 고친다. 그쪽이 먼저 머지되면 이 브랜치를 갱신하면서 그 PR 이 넣는 app 관련 refactor(platform): 알림 권한 ViewModel과 Effect를 MVI로 전환 #1809 주석도 함께 정리한다.