Skip to content

✅ Test - [머지 금지] 스크린샷 비교 CI 동작 확인용 칩 모양 변경 - #159

Draft
DongChyeon wants to merge 1 commit into
developfrom
chore/#153-screenshot-comparison-smoke-test
Draft

DongChyeon wants to merge 1 commit into
developfrom
chore/#153-screenshot-comparison-smoke-test

Conversation

@DongChyeon

Copy link
Copy Markdown
Member

⚠️ 머지하지 마세요. 스크린샷 비교 CI 가 실제로 동작하는지 확인하기 위한 일회용
PR 입니다. 확인이 끝나면 닫고 브랜치를 지웁니다.

🛠 Related issue

#153 검증용 (closes 하지 않습니다)

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

✅ CheckPoint

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

✏️ Work Description

  • BuyOrNotChip / BuyOrNotIconChipshapeCircleShapeRoundedCornerShape(4.dp)
    로 바꿔 의도적으로 스크린샷 diff 를 만듭니다.
  • 파일 하나(Chip.kt), 3줄짜리 변경이라 되돌리기 쉽습니다.

😅 Uncompleted Tasks

  • 확인이 끝나면 이 PR 을 닫고 브랜치를 삭제합니다. 칩 모양은 원래대로 원형이어야 합니다.

📢 To Reviewers

왜 여는가

#153 으로 스크린샷 비교 CI 를 넣었지만, 지금까지 비교 경로가 한 번도 실제로 돌아본 적이
없습니다. #157 (콜론 문제) 때문에 골든 아티팩트가 만들어지지 않아 모든 PR 이
"기준선 없음" 으로 스킵됐기 때문입니다.

#158 이 머지되면서 골든이 처음으로 생성됐습니다.
(run 35480621139,
아티팩트 screenshot-goldens 2.4MB)

이 PR 은 그 기준선에 대고 실제로 diff 가 잡히는지, 코멘트가 달리는지, 컴패니언 브랜치에
이미지가 올라가 렌더되는지를 확인합니다.

무엇을 볼 것인가

  • Screenshot comparison 잡이 골든을 내려받아 compareRoborazziDebug 를 돌리는지
  • 봇이 diff 표가 든 코멘트를 남기는지 (표 안 이미지가 실제로 보이는지)
  • 칩을 쓰는 화면까지 번지는지

로컬에서 같은 조건으로 미리 돌려본 결과는 프리뷰 7개였습니다. 칩 자체 2개에
Home / MyPage / Notification 화면이 더해집니다. CI 결과가 이와 다르면 그 차이 자체가
살펴볼 거리입니다.

리뷰 부담 없습니다 (A)

코드를 평가할 PR 이 아닙니다. CI 가 남기는 코멘트만 같이 봐주시면 됩니다.

📃 RCA 룰

  • R: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
  • C: 웬만하면 반영해 주세요. (Comment)
  • A: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

칩을 원형에서 4dp 라운드 사각형으로 바꿔 스크린샷 diff 를 의도적으로 발생시킨다.
비교 워크플로가 실제로 diff 를 잡아 PR 코멘트로 보고하는지 확인하기 위한 것이며,
검증이 끝나면 되돌린다.
@coderabbitai

coderabbitai Bot commented Sep 20, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

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.

1 participant