-
Notifications
You must be signed in to change notification settings - Fork 26
chore: upgrade opensumi to 3.6.5 #288
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
Conversation
Ricbet
left a comment
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
📝 WalkthroughWalkthrough这是一个版本更新 PR,将单体仓库项目版本从 2.4.4 更新到 2.4.6,同时将 OpenSumi 依赖版本从 3.6.3 更新到 3.6.5-next-1766394426.0。 Changes
预估代码审查工作量🎯 2 (Simple) | ⏱️ ~8 分钟
相关的可能 PR
建议审查人
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 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 |
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: 1
📜 Review details
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (14)
lerna.jsonpackage.jsonpackages/cli/package.jsonpackages/code-api/package.jsonpackages/code-service/package.jsonpackages/common/package.jsonpackages/core/package.jsonpackages/i18n/package.jsonpackages/plugin/package.jsonpackages/registry/package.jsonpackages/startup/package.jsonpackages/sumi-core/package.jsonpackages/toolkit/define.jsonpackages/toolkit/package.json
🔇 Additional comments (17)
lerna.json (1)
3-3: 版本更新正确Lerna 版本已正确更新至 2.4.6。
packages/common/package.json (1)
3-3: 版本更新正确版本已正确更新至 2.4.6,与 monorepo 整体版本保持一致。
packages/sumi-core/package.json (2)
3-3: 版本更新正确版本已正确更新至 2.4.6。
19-26: 确认 OpenSumi 3.6.5-next-1766394426.0 版本是否已修复提及的问题OpenSumi 依赖已更新至 3.6.5-next-1766394426.0(预发布版本)。需要确认:
- 此版本已通过充分测试
- 确实修复了 PR 中提到的问题(切换文件时评论组件不显示、粘贴图片上传失败)
packages/i18n/package.json (1)
3-3: 版本更新正确版本已正确更新至 2.4.6。
packages/code-service/package.json (1)
3-3: 版本更新正确版本已正确更新至 2.4.6。
packages/code-api/package.json (1)
3-3: 版本更新正确版本已正确更新至 2.4.6。
packages/registry/package.json (1)
3-3: 版本更新正确版本已正确更新至 2.4.6。
packages/plugin/package.json (1)
3-3: 版本更新看起来正确。版本号从 2.4.4 升级到 2.4.6,与整个 monorepo 的版本对齐一致。
packages/startup/package.json (1)
3-3: 版本更新正确。版本号升级到 2.4.6,与其他包保持一致。
package.json (3)
10-10: 确认 OpenSumi 引擎版本更新。引擎版本从 3.6.3 更新到预发布版本 3.6.5-next-1766394426.0。请确保此版本与 packages/toolkit/define.json 和其他包中的依赖版本保持一致。
89-90: 新增的 packageManager 和 version 字段。根目录 package.json 新增了这两个字段:
packageManager: 锁定 yarn 版本为 4.2.2version: 设置为 2.4.6请确认:
- 在 monorepo 根目录添加 version 字段是否符合预期(通常 Lerna 项目不在根目录设置 version)
- 团队是否准备好迁移到 Yarn 4.2.2
10-90: PR 描述与代码变更不一致。PR 描述提到修复了两个 bug(评论组件不显示、图片上传失败),但所有可见的代码变更都是版本号更新,没有看到实际的 bug 修复代码。
可能的原因:
- Bug 修复在 OpenSumi 上游仓库中完成,此 PR 仅更新依赖版本
- 修复的代码文件未包含在此次审查范围内
请确认 bug 修复确实包含在 OpenSumi 3.6.5-next-1766394426.0 版本中。
packages/cli/package.json (1)
3-3: 版本和依赖更新正确。包版本、引擎版本和 OpenSumi 依赖版本都已正确更新到最新版本,保持一致性。
Also applies to: 13-13, 19-19
packages/core/package.json (2)
3-3: 版本更新正确。包版本升级到 2.4.6,与 monorepo 其他包保持一致。
36-77: 批量更新 OpenSumi 依赖版本。所有 42 个 @opensumi/* 依赖已更新到 3.6.5-next-1766394426.0。这是一个大规模的依赖更新,建议合并前:
- 验证所有依赖包在该版本下都可用
- 确保 yarn.lock 或 pnpm-lock.yaml 文件已正确更新
- 进行全面的集成测试,特别是验证评论组件和图片上传功能
packages/toolkit/define.json (1)
2-2: 验证 OpenSumi 预发布版本 3.6.5-next-1766394426.0 的可用性。此版本是 OpenSumi 的预发布快照版本。确认以下事项:
- 该版本号格式有效,符合 npm 预发布版本命名规范
- OpenSumi 确实发布 "next" 标签的预发布版本,但需确认此特定版本是否已在 npm registry 中发布
- 预发布版本不建议用于生产环境——应使用稳定版本(当前最新为 3.9.0)
建议:若无特殊原因,请使用最新稳定版本 3.9.0 或针对 3.6.5 需求的最新稳定补丁版本。
Ricbet
left a comment
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
Types
Background or solution
ChangeLog
修复切换文件时评论组件不显示的问题
修复粘贴图片上传失败的问题
Summary by CodeRabbit
发布说明
✏️ Tip: You can customize this high-level summary in your review settings.