fix(ci): corepack manager use pnpm failed#2970
Conversation
Walkthrough此次 PR 修改了多个 GitHub Actions 工作流文件中的 pnpm 安装命令。原先使用 “corepack enable pnpm” 启用 pnpm，如今全部替换为 “npm i -g pnpm” 以全局安装。受影响的文件包括 ci.yml、realease-tag.yml、sync-h5.yml、sync-migrate.yml 和 sync-taro.yml。除安装命令外，工作流的其他步骤、触发条件及导出的公共实体均保持不变。 Changes
Possibly related PRs
Suggested reviewers
Poem
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 (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #2970 +/- ##
=======================================
Coverage 84.19% 84.19%
=======================================
Files 267 267
Lines 18099 18099
Branches 2665 2665
=======================================
Hits 15238 15238
Misses 2856 2856
Partials 5 5 ☔ View full report in Codecov by Sentry. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (5)
.github/workflows/realease-tag.yml (1)
17-17: 确认pnpm安装命令的修改
此处将原来的 “corepack enable pnpm” 替换为 “npm i -g pnpm”，采用全局安装方式来确保 pnpm 可用，能够解决 Corepack 相关的问题。建议后续如有版本稳定性需求，可考虑指定安装版本。.github/workflows/sync-h5.yml (1)
20-20: 全局安装pnpm命令确认
将 “corepack enable pnpm” 修改为 “npm i -g pnpm” 能够确保在 CI 环境中正确安装 pnpm，支持后续依赖安装和构建任务。建议后续关注 pnpm 版本问题以避免潜在不一致性。.github/workflows/sync-taro.yml (1)
20-20: 确认pnpm安装命令更新
此处采用 "npm i -g pnpm" 替换旧的 Corepack 命令，保证环境中 pnpm 的全局安装，从而确保后续文档同步和依赖安装流程正常运行。.github/workflows/sync-migrate.yml (1)
21-21: 更新全局安装pnpm命令
使用 "npm i -g pnpm" 替换原有的 Corepack 命令，确保在迁移同步任务中全局安装 pnpm，有助于提升 CI 流程的稳定性。建议后续检查是否需要对 pnpm 版本进行锁定。.github/workflows/ci.yml (1)
64-66: 在 Build 任务中确认pnpm安装命令
在构建流程中采用 “npm i -g pnpm” 命令，确保 pnpm 全局安装，从而维持依赖安装和构建步骤的稳定性。建议注意与缓存及版本管理相关的潜在问题。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
.github/workflows/ci.yml(3 hunks).github/workflows/realease-tag.yml(1 hunks).github/workflows/sync-h5.yml(1 hunks).github/workflows/sync-migrate.yml(1 hunks).github/workflows/sync-taro.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
🔇 Additional comments (2)
.github/workflows/ci.yml (2)
20-21: 在 Lint 任务中确认pnpm安装命令
将 “corepack enable pnpm” 修改为 “npm i -g pnpm”，确保在 Lint 流程中全局安装 pnpm，从而避免因 Corepack 导致的安装失败问题。
39-41: 在 Test 任务中确认pnpm安装命令
此修改确保在测试流程中 pnpm 能够全局安装并正常使用，保障后续依赖安装和测试执行的一致性。
Miles-hxy
changed the title
3 participants
🤔 这个变动的性质是？
🔗 相关 Issue
💡 需求背景和解决方案
☑️ 请求合并前的自查清单
Summary by CodeRabbit