-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(all): Optimize packages #208
Conversation
- 전체 패키지를 배포하고 있던 기존의 번들을 dist 폴더와 variable.css만 배포하여 번들사이즈를 줄입니다.
|
🦋 Changeset detectedLatest commit: d4630ee The changes in this PR will be included in the next version bump. This PR includes changesets to release 5 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
"@vanilla-extract/sprinkles": "1.6.1", | ||
"tsup": "^8.0.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deDepedancy에 이미 tsup이 존재하기 때문에 dependacy에서 제거했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Vitest랑 VE/sprinkles도 캐럿버저닝으로 쓰면 좋을 것 같네용~~~
packages/ui/package.json
Outdated
@@ -22,8 +28,7 @@ | |||
"@sopt-makers/fonts": "workspace:^", | |||
"@sopt-makers/icons": "workspace:^", | |||
"@vanilla-extract/css": "^1.14.0", | |||
"@vanilla-extract/sprinkles": "1.6.1", | |||
"tsup": "^8.0.2" | |||
"@vanilla-extract/sprinkles": "1.6.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 캐럿버저닝으로 쓰는 게 좋아보이네용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
변경사항
링크
시급한 정도
🏃♂️ 보통 : 최대한 빠르게 리뷰 부탁드립니다.
기타 사항