✅ Test - [머지 금지] 스크린샷 비교 CI 동작 확인용 칩 모양 변경 - #159
Draft
DongChyeon wants to merge 1 commit into
Draft
DongChyeon wants to merge 1 commit into
DongChyeon wants to merge 1 commit into
Conversation
칩을 원형에서 4dp 라운드 사각형으로 바꿔 스크린샷 diff 를 의도적으로 발생시킨다. 비교 워크플로가 실제로 diff 를 잡아 PR 코멘트로 보고하는지 확인하기 위한 것이며, 검증이 끝나면 되돌린다.
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
스크린샷 비교 리포트 프리뷰 7개의 렌더가 base 와 다릅니다. 의도한 변경인지 확인해주세요.
|
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.






🛠 Related issue
#153 검증용 (closes 하지 않습니다)
어떤 변경사항이 있었나요?
✅ CheckPoint
PR이 다음 요구 사항을 충족하는지 확인하세요.
✏️ Work Description
BuyOrNotChip/BuyOrNotIconChip의shape를CircleShape→RoundedCornerShape(4.dp)로 바꿔 의도적으로 스크린샷 diff 를 만듭니다.
Chip.kt), 3줄짜리 변경이라 되돌리기 쉽습니다.😅 Uncompleted Tasks
📢 To Reviewers
왜 여는가
#153 으로 스크린샷 비교 CI 를 넣었지만, 지금까지 비교 경로가 한 번도 실제로 돌아본 적이
없습니다. #157 (콜론 문제) 때문에 골든 아티팩트가 만들어지지 않아 모든 PR 이
"기준선 없음" 으로 스킵됐기 때문입니다.
#158 이 머지되면서 골든이 처음으로 생성됐습니다.
(run 35480621139,
아티팩트
screenshot-goldens2.4MB)이 PR 은 그 기준선에 대고 실제로 diff 가 잡히는지, 코멘트가 달리는지, 컴패니언 브랜치에
이미지가 올라가 렌더되는지를 확인합니다.
무엇을 볼 것인가
Screenshot comparison잡이 골든을 내려받아compareRoborazziDebug를 돌리는지로컬에서 같은 조건으로 미리 돌려본 결과는 프리뷰 7개였습니다. 칩 자체 2개에
Home / MyPage / Notification 화면이 더해집니다. CI 결과가 이와 다르면 그 차이 자체가
살펴볼 거리입니다.
리뷰 부담 없습니다 (A)
코드를 평가할 PR 이 아닙니다. CI 가 남기는 코멘트만 같이 봐주시면 됩니다.
📃 RCA 룰