fix(deps): use workspace:^ for internal package dependencies - #681
fix(deps): use workspace:^ for internal package dependencies#681MaxLee-dev wants to merge 1 commit into
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: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. 📝 WalkthroughWalkthrough워크스페이스 의존성 범위를 Changes워크스페이스 의존성 범위 조정
Estimated code review effort: 1 (매우 간단) | ~2분 Merge Risk: ⚪ Minimal · up to This PR updates internal package dependency ranges and the lockfile to preserve semver ranges in published packages; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
✅ All tests passed!
Click here if you need to update snapshots. |
Problem
워크스페이스 내부 패키지 의존성이
workspace:*로 고정되어 있었습니다. 배포된 버전에는 workspace:*가 아닌 고정된 버전으로 들어가게 됩니다.예시 - @vapor-ui/core@1.5.0 package.json
이는 changeset에서 패치를 통해 릴리즈에 포함되어 있지 않은 종속 패키지도 함께 추가해서 버전이 변경되도록 설정되어 있기 때문.
Solution
@vapor-ui/hooks,@vapor-ui/icons,@vapor-ui/color-generator의존성 명시자를workspace:^로 변경해 배포 시 semver 범위가 적용되도록 했다.pnpm-lock.yaml도 함께 갱신했다.Summary by CodeRabbit