Skip to content

refactor: dialog 애니메이션 수정 - #172

Merged
KwonDeaGeun merged 1 commit into
developfrom
refactor/modify-dialog-animation
Feb 9, 2026
Merged

refactor: dialog 애니메이션 수정#172
KwonDeaGeun merged 1 commit into
developfrom
refactor/modify-dialog-animation

Conversation

@KwonDeaGeun

@KwonDeaGeun KwonDeaGeun commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

릴리스 노트

  • Style
    • UI 컴포넌트에서 코드 포맷팅을 개선했습니다. 이는 기능이나 동작에 영향을 주지 않습니다.

@KwonDeaGeun KwonDeaGeun self-assigned this Feb 9, 2026
@coderabbitai

coderabbitai Bot commented Feb 9, 2026

Copy link
Copy Markdown

개요

src/components/ui/dialog.tsx 파일의 cn(...) 호출에서 className 인자 뒤에 후행 쉼표가 추가되었습니다. 런타임 동작이나 제어 흐름에는 영향을 주지 않는 서식 변경입니다.

변경 사항

응집 / 파일 요약
UI 대화 상자 서식 개선
src/components/ui/dialog.tsx
cn(...) 호출에서 className 인자 뒤에 후행 쉼표 추가. 함수형 동작이나 내보낸 인터페이스에는 영향 없음.
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning PR 제목은 '애니메이션 수정'을 언급하지만 실제 변경사항은 cn() 호출의 후행 쉼표 포맷팅 변경으로, 애니메이션과 무관합니다. 제목을 실제 변경사항에 맞게 수정하세요. 예: 'refactor: dialog 컴포넌트 포맷팅 개선' 또는 'refactor: cn() 호출에 후행 쉼표 추가'
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉


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 and usage tips.

@KwonDeaGeun
KwonDeaGeun merged commit 0d0929b into develop Feb 9, 2026
2 checks passed
@KwonDeaGeun
KwonDeaGeun deleted the refactor/modify-dialog-animation branch February 9, 2026 04:15
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