Skip to content

Conversation

@hyoseong1994
Copy link
Contributor

변경 사항

  • radix-ui 디펜던시 제거
  • chromatic, axe-playwright 디펜던시 -> devDependency 로 이동

목적

  • npm에 빌드된 v0.0.5를 보다가 불필요한 디펜던시가 있고 dev에 들어가야할게 들어있는걸 발견했습니다.

리뷰어에게

PR 작성자 체크 리스트

  • UI Review를 요청하고 검토를 받았나요?
  • UI Tests를 진행했나요?

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Jan 8, 2026

Bundle Report

Bundle size has no change ✅

@hyoseong1994 hyoseong1994 changed the title Refactor dependency 불필요한 디펜던시 제거 및 dev 로 이동 Jan 8, 2026
Copy link
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이런 부분까지 챙겨주시고 최고에용! 👍

Comment on lines -9 to -10
"@radix-ui/react-checkbox": "^1.3.3",
"@radix-ui/react-radio-group": "^1.3.8",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

헐, 얘네가 아직도 있었어요? ㅋㅋ 마이그레이션 하면서 제거된 줄 알았네요.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 제거된지 알았는데 npm에서 있는거 보고 놀랐습니다 ㅎㅎ;;

@hyoseong1994 hyoseong1994 added this pull request to the merge queue Jan 8, 2026
Merged via the queue into main with commit 45d4161 Jan 8, 2026
15 checks passed
@hyoseong1994 hyoseong1994 deleted the refactor-dependency branch January 8, 2026 23:41
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.

3 participants