-
Notifications
You must be signed in to change notification settings - Fork 280
fix: corepack manager use pnpm failed #2962
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
Caution Review failedThe pull request is closed. Walkthrough此 PR 对 CI 工作流、项目构建脚本、样式配置及文档进行了大规模更新。主要修改包括:在各个 CI 工作流中将 pnpm 的安装命令从单条命令调整为多行命令,明确指定版本;调整 package.json 中的脚本命令;更新部分文档内容、格式以及组件属性;重构站点配置和路由;并新增多个 React 版文档组件及 Vite 配置文件,从而优化构建流程和项目结构。 Changes
Sequence Diagram(s)sequenceDiagram
participant Job as CI 作业
participant Corepack as Corepack 工具
participant NPM as npm
Job->>Corepack: 启动并启用 Corepack
alt lint/test 作业
Corepack->>Job: 执行 "corepack enable" 后准备 [email protected] 激活
else build 作业
Corepack->>Job: 禁用 Corepack
Job->>NPM: 执行 npm install -g pnpm@latest
end
Possibly related PRs
Suggested reviewers
Poem
✨ Finishing Touches
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 (
|
pnpm/pnpm#9029 (comment)
近期corepack使用pnpm类似报错,参考社区的解决方案
Summary by CodeRabbit
新功能
文档
样式
其他