Skip to content

feat: update Copilot setup to use pnpm for dependency management#61

Merged
chenyme merged 1 commit intolinux-do:masterfrom
OrenZhang:chore_pnpm_copilot
Jul 6, 2025
Merged

feat: update Copilot setup to use pnpm for dependency management#61
chenyme merged 1 commit intolinux-do:masterfrom
OrenZhang:chore_pnpm_copilot

Conversation

@OrenZhang
Copy link
Contributor

例行检查

  • 我已阅读并理解 贡献者公约
  • 我已阅读并同意 贡献者许可协议 (CLA),确认我的贡献将根据项目的 MIT 许可证进行许可,
  • 我知晓如果此 PR 并不做出实质性更改,或可被认为是为了PR被合并而提交PR的,则可能不会被合并,

变更内容

修改为 PNPM 管理包

@OrenZhang OrenZhang self-assigned this Jul 6, 2025
Copilot AI review requested due to automatic review settings July 6, 2025 04:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

本次 PR 目的是将 Copilot 工作流从 npm 切换为 pnpm 进行依赖管理

  • 新增 “Enable pnpm” 步骤,使用 Corepack 启用并激活 pnpm
  • 更新 actions/setup-node 缓存类型为 pnpm,并将锁文件路径指向 frontend/pnpm-lock.yaml
  • 将依赖安装命令由 npm install 替换为 pnpm install
Comments suppressed due to low confidence (2)

.github/workflows/copilot-setup-steps.yml:23

  • [nitpick] 建议使用官方的 pnpm/action-setup Action 来安装并管理 pnpm 版本,以减少对 corepack 命令的依赖并更清晰地控制版本。
      - name: Enable pnpm

.github/workflows/copilot-setup-steps.yml:35

  • 建议在安装依赖时添加 --frozen-lockfile 标志,确保工作流使用锁文件中的精确版本,从而提高构建一致性和可重复性。
        run: pnpm install

@chenyme chenyme self-requested a review July 6, 2025 04:45
@chenyme chenyme merged commit 9e026ef into linux-do:master Jul 6, 2025
11 checks passed
@OrenZhang OrenZhang deleted the chore_pnpm_copilot branch July 6, 2025 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants