Skip to content

refactor(core): app·core·afternote·receiver·setting 의 runCatchingCancellable 을 core:domain 경로로 바꾼다 (#2267) - #2269

Merged
OnlyReadOneBook merged 2 commits into
developfrom
1hyok/friendly-wright-nwzebe
Oct 5, 2026
Merged

OnlyReadOneBook merged 2 commits into
developfrom
1hyok/friendly-wright-nwzebe

Conversation

@OnlyReadOneBook

Copy link
Copy Markdown
Collaborator

먼저 feature/setting/domain/.../UpdateReceiverInfoUseCase.kt 를 읽어 주세요. 나머지 31개 파일은 import 한 줄씩만 바뀝니다.

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

검증

  • 단위 테스트: setting domain 10, setting presentation 132, afternote data 99, afternote presentation 341, receiver data 107, core data 109 전부 통과
  • 대상 모듈 ktlintCheck, :app:compileDebugKotlin, :app:compileDebugUnitTestKotlin, :app:compileDebugAndroidTestKotlin, :konsist:test(56건) 통과

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "none",
    "reason": "같은 함수의 import 경로만 바꾸고 UseCase 한 곳의 try/catch 를 같은 동작의 함수 호출로 바꿨다. 화면·네비게이션·매니페스트·androidTest 소스 변경이 없고, 바뀐 UseCase 동작은 JVM 단위 테스트가 잠근다.",
    "tests": []
  }
}

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

화면 변경 없음

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

core:common 의 Deprecated 위임 함수는 다른 모듈이 아직 쓰고 있어 이 PR 에서 지우지 않습니다.


Generated by Claude Code

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

열린 PR 중 이 PR 과 같은 파일을 서로 다르게 고친 것이 있다. 어느 쪽이 먼저 머지되든 나중 쪽은 base 를 병합해야 통과한다 (#1750).

머지 자체는 GitHub 이 막는다 — 한쪽이 들어가면 다른 쪽은 CONFLICTING 이 되어 merge queue 에 못 들어간다.
이 라벨은 그 전에 순서를 정하라는 신호다. 먼저 갈 쪽을 정하고, 나중 쪽은 그 머지 뒤에 base 를 병합한다.
상대 PR 에도 같은 안내가 달린다.

이 코멘트는 리컨사일러가 매 실행 현재 상태로 고쳐 쓴다. 해소되면 sibling-conflict 라벨은 자동으로 떨어진다.

@OnlyReadOneBook
OnlyReadOneBook added this pull request to the merge queue Oct 5, 2026
Merged via the queue into develop with commit 725c8b7 Oct 5, 2026
26 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the 1hyok/friendly-wright-nwzebe branch October 5, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(core): app·core·afternote·receiver·setting 의 runCatchingCancellable import 를 core:domain 경로로 바꾼다

2 participants