Skip to content
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

refactor: change package manager to pnpm #4974

Conversation

zhangpaopao0609
Copy link
Collaborator

@zhangpaopao0609 zhangpaopao0609 commented Jan 31, 2025

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

#4975

💡 需求背景和解决方案

将包管理器修改为 pnpm

  1. package.json -> packageManager + script use pnpm
  2. package.json -> 更新 build:tsc,不清楚这与 pnpm 有没有关系,但如果不改,actions ci 会出错
    https://github.com/zhangpaopao0609/tdesign-vue-next/actions/runs/13068725037/job/36465574592
image

相关解决办法:https://github.com/pnpm/pnpm/issues/6291,将 npm-run-all 更新为 npm-run-all2

  1. 更新 ci
    主要是将 npm run 改为 pnpm run,因为大部分 yml 文件使用的是 tdesign 的,因此这里暂时只在本项目改了 pull-request.yml,具体的如何更改 @uyarn 需要讨论一下

📝 更新日志

  • refactor: change package manager to pnpm

  • 本条 PR 不需要纳入 Changelog

@zhangpaopao0609
Copy link
Collaborator Author

这里的 ci 过不了是因为 ci 是用的 tdesign 的 yml 文件,而本次更改又需要更新 ci 才可以

@zhangpaopao0609 zhangpaopao0609 changed the title Feature/paopao/package manager change to pnpm refactor: change package manager to pnpm Jan 31, 2025
@zhangpaopao0609 zhangpaopao0609 mentioned this pull request Jan 31, 2025
16 tasks
Copy link
Contributor

github-actions bot commented Jan 31, 2025

完成

@zhangpaopao0609
Copy link
Collaborator Author

zhangpaopao0609 commented Jan 31, 2025

这里的 ci 过不了是因为 ci 是用的 tdesign 的 yml 文件,而本次更改又需要更新 ci 才可以

@uyarn
感觉不是很符合常理,我预期应该是过不了 ci 的,原因如上。但过了,发现 ci 居然用的是我 pr 里面的 pull-request.yml,这样不会有问题吗?

@zhangpaopao0609 zhangpaopao0609 self-assigned this Jan 31, 2025
@zhangpaopao0609 zhangpaopao0609 added the monorepo monorepo label Jan 31, 2025
@liweijie0812
Copy link
Collaborator

正常的, pr 跑的ci 是使用当前分支的文件

@zhangpaopao0609
Copy link
Collaborator Author

更新上述第 2 点:
相关解决办法:pnpm/pnpm#6291 npm-run-all 更新为 npm-run-all2

@zhangpaopao0609
Copy link
Collaborator Author

正常的, pr 跑的ci 是使用当前分支的文件

好吧,只是第一眼觉得不符合常理,因为这里 ci 的目的是为了前置检查 pr 的代码是否符合规范,结果用的是本分支的 ci,那我胡乱改,只要本分支 ci 能过,那么在 pr 的 ci 里看都是正常

@uyarn
Copy link
Collaborator

uyarn commented Jan 31, 2025

正常的, pr 跑的ci 是使用当前分支的文件

好吧,只是第一眼觉得不符合常理,因为这里 ci 的目的是为了前置检查 pr 的代码是否符合规范,结果用的是本分支的 ci,那我胡乱改,只要本分支 ci 能过,那么在 pr 的 ci 里看都是正常

正常情况下,不会随意合入非长期维护同学对CI改动的PR;改动了CR过程也可以看到改动点

@uyarn uyarn merged commit b1a8cd8 into Tencent:develop Feb 1, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monorepo monorepo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants