Repository navigation
Conversation
🚧 리뷰 적체 가드 — 이 PR 을 닫습니다아무도 리뷰하지 않은 PR 이 4건 남아 있습니다.
이 저장소는 아무도 응답하지 않은 PR 이 1건이라도 남아 있으면 새 PR 을 열 수 없습니다 (#1136). 각 PR 에 필요한 리뷰는 한 건입니다 — 누구든 먼저 판정을 내면 그 PR 은 모두의 목록에서 빠집니다. 승인 뒤에 올라온 변경을 다시 봐 줄 의무는 없지만, 변경요청은 작성자가 반영한 시점에 다시 리뷰어의 몫이 됩니다. 되살리는 법위 리뷰를 끝낸 뒤 이 PR 을 다시 여세요. 브랜치는 지워지지 않았고 커밋도 그대로입니다. gh pr reopen 1198리뷰보다 이 PR 이 급하면 |
|
❌ Screenshot test fail — baseline 과 CI rendered PNG 차이. 📦 Artifact: https://github.com/Afternote/Afternote-FE/actions/runs/32951187023 (페이지 하단 Artifacts → 🔍 확인 위치 (artifact zip 내부):
📝 조치 가이드:
CI ↔ 로컬 환경 차이는 본 워크플로의 container 단계가 root fix 한다 (README 의 "Compose Preview Screenshot Testing (docker baseline)" 섹션 참조). |
#1174 가 `AfternoteTypography.kt` 에 lineBreak·ko 로케일을 넣어 한글이 어절 단위로 끊기게 바뀌었다. 같은 날 들어간 #1162·#1164 의 baseline 은 그 이전 렌더라, develop 조합에서 `validateDebugScreenshotTest` 가 세 화면에서 ImageComparisonAssertionError 를 낸다 — 약관 화면 Compact, 수신자 전달완료·이메일인증 fontScale 1.5. CI 와 같은 컨테이너(amd64·ubuntu 24.04·Noto CJK)에서 다시 떴다. 약관 화면은 「(선택)」이 「(선」/「택)」으로 쪼개지던 것이 통째로 내려간다 — #1142 가 고치려던 그 증상이다.
|
같은 사고를 다른 세션에서 독립적으로 추적하다 이 PR 과 중복되는 복구를 만들었습니다(#1200 · #1204). 이 PR 이 먼저이고 범위도 더 넓어 제 쪽을 닫았습니다. 닫기 전에 대조해 두었는데, 참고가 될 것 같아 남깁니다 — baseline 3장이 바이트 단위로 동일합니다. 서로 다른 워크트리·별도 gradle 캐시 볼륨에서 각자 docker( 추가로 artifact 의 rendered 와 대조해 문구는 동일하고 줄바꿈 위치만 다름을 확인했습니다. 즉 현재 baseline 이 #1142 가 고치려던 결함(한글 어절 중간 줄바꿈)을 정답으로 굳혀 두고 있던 것이고, 이 PR 의 갱신 방향이 맞습니다.
🤖 Addressed by Claude Code |
develop 이 컴파일도, 스크린샷 검증도 통과하지 못하는 상태다. 원인이 둘인데 성격이 같다 — 오늘 머지된 PR 들이 서로 파일을 하나도 겹치지 않으면서 조합에서만 깨졌다.
merge-tree로도 clean, 각자 CI 도 green 이었다.1. 컴파일 — 리소스 개명 ↔ 옛 이름 참조
#1187 이 onboarding 리소스를
onboarding_프리픽스로 개명했고, #1162 는 옛 이름을 참조하는 테스트를 새로 추가했다. 참조 두 줄을 새 이름으로 옮긴다 — 테스트가 무엇을 보는지는 그대로다.이 실패는 develop 머지 직후 워밍 run 32950531237 이 이미 잡았다.
build-cache-warm.yml은 게이트가 아니라 실패가 아무 데도 도달하지 않았을 뿐이다.2. 스크린샷 — 타이포그래피 변경 ↔ 그 이전 baseline
#1174 가
AfternoteTypography.kt에lineBreak·ko 로케일을 넣어 한글이 어절 단위로 끊기게 바뀌었다(#1142). #1162·#1164 의 baseline 은 그 이전 렌더라 어긋난다. CI 와 같은 컨테이너(amd64·ubuntu 24.04·Noto CJK)에서 세 장을 다시 떴다.약관 화면은 개선이 눈에 보인다 — 「E-mail 및 SMS 광고성 정보 수신동의 (선택)」이
(선/택)으로 쪼개지던 것이 이제(선택)통째로 내려간다. #1142 가 고치려던 그 증상이다.검증
./gradlew compileDebugUnitTestKotlin compileDebugAndroidTestKotlin— BUILD SUCCESSFUL. 전 모듈 26개 태스크 +:app:compileDebugAndroidTestKotlin. 수정 전 같은 명령이 위 오류로 BUILD FAILED 였다.validateDebugScreenshotTest(onboarding·receiver) — BUILD SUCCESSFUL. CI green 사전 확정.R.string.*참조를strings.xml정의와 전수 대조 — 정의에 없는 참조 0건.리뷰어에게
이 자리는 다시 열린다.
resourcePrefix도입(#1071·#1185)이 모듈별로 진행 중이고, 타이포그래피·공용 컴포넌트 변경도 계속 있다. 정의를 바꾸는 PR 과 그 이름을 새로 참조하는 PR 이 같은 배치에 있으면 파일 충돌 없이 조합에서만 깨진다 — GitHub 의 MERGEABLE 도merge-tree도 이 축을 보지 못한다.develop push 에는 검증 워크플로가 없어서(
pr-validation.yml은pull_request전용) 이 red 가 자동으로는 어디에도 뜨지 않는다. 열린 PR 들이 실패를 물려받고서야 간접적으로 드러나고, 그때조차 «내 PR 탓인지 develop 탓인지» 를 사람이 로그를 갈라 판정해야 한다(#1176 에서 실제로 그렇게 갈랐다). 별도 이슈로 다룰 만하다.이 PR 이 두 갈래를 함께 닫는다. 같은 시각 다른 세션에서 #1199(컴파일)·#1201(baseline)로 나뉘어 진단된 것과 원인·범위가 같고, 이쪽은 두 축 모두 수정과 검증이 끝나 있다.
Closes #1199
Closes #1201