-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
fix: resolve ghost dependency issue for strict pnpm compatibility #223
Conversation
Fixed an issue where ghost dependencies prevented the project from starting correctly under pnpm's strict mode.
📝 Walkthrough📝 WalkthroughWalkthrough
Changes
Possibly related PRs
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🔇 Additional comments (4)package.json (4)
与现有的 @emotion/server 依赖配合使用,用于支持 CSS-in-JS 功能。版本选择合适。
之前的评论关于重复的剪贴板功能依赖的问题仍然有效。
之前的评论关于移除未使用的 rc-drawer 依赖的建议仍然有效。 Line range hint 建议验证新添加的依赖是否解决了 ghost dependency 问题,并确保在所有相关的工作区中都正确声明了这些依赖。 Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Deploying ant-design-x with Cloudflare Pages
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #223 +/- ##
=======================================
Coverage 91.13% 91.13%
=======================================
Files 66 66
Lines 1422 1422
Branches 379 379
=======================================
Hits 1296 1296
Misses 126 126 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
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.
Actionable comments posted: 2
🧹 Outside diff range and nitpick comments (1)
package.json (1)
Line range hint
103-223
: 建议:考虑对新增依赖进行打包分析为了确保这些新增依赖不会显著增加打包体积,建议进行打包分析。
可以使用项目中已配置的 webpack-bundle-analyzer 进行分析。同时也建议检查是否所有新增依赖都被正确地打包和 tree-shaking。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (1)
bun.lockb
is excluded by!**/bun.lockb
📒 Files selected for processing (1)
package.json
(3 hunks)
🔇 Additional comments (1)
package.json (1)
103-103
: 添加 @emotion/react 依赖
这是一个合理的添加,因为 @emotion/react 经常作为 CSS-in-JS 解决方案的核心依赖。建议验证项目中的实际使用情况。
Bundle ReportBundle size has no change ✅ |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected] |
Fixed an issue where ghost dependencies prevented the project from starting correctly under pnpm's strict mode.
Summary by CodeRabbit