refactor: JDS Code 컴포넌트 vanilla-extract 마이그레이션#438
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughCode 컴포넌트를 Emotion 기반 스타일에서 vanilla-extract 기반 스타일로 전환합니다. 새로운 타입 정의와 CSS recipe를 도입하고, 컴포넌트를 forwardRef로 리팩토링하여 네이티브 ChangesCode 컴포넌트 vanilla-extract 마이그레이션
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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. Comment |
itwillbeoptimal
left a comment
There was a problem hiding this comment.
확인이 늦었습니다 🥲 소희님께서 이미 대부분 언급해주신 것 같아서 추가로 코멘트 드릴 부분은 없는 것 같습니다~ 고생하셨습니다
| import { recipe } from "@vanilla-extract/recipes"; | ||
|
|
||
| import type { CodeSize } from "./code.types"; | ||
| import { vars } from "../../tokens/vars.css"; |
There was a problem hiding this comment.
(단순 의견) 마이그레이션 진행하면서 ../../처럼 부모 경로가 여러 단계 중첩되는 import가 종종 보이는데, 가독성이나 경로 관리 측면에서 단축 경로로 통일하는 것도 괜찮을 것 같습니다
import { vars } from "tokens";
💡 작업 내용
Codevanilla-extract 마이그레이션💡 자세한 설명
+) 파일명 대소문자 관련하여 질문드렸을 때 ts 파일은 소문자로 통일하는 것이 좋겠다는 답변을 받아 파일명 수정하였으니 확인 부탁드립니다.
📗 참고 자료 (선택)
📢 리뷰 요구 사항 (선택)
✅ 셀프 체크리스트
closes #434
Summary by CodeRabbit