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

ci: adjust MAIN_PULL_REQUEST #4972

Closed
wants to merge 8 commits into from
Closed

Conversation

liweijie0812
Copy link
Collaborator

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

  • fix(组件名称): 处理问题或特性描述 ...

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

Copy link
Contributor

github-actions bot commented Jan 27, 2025

失败

@liweijie0812
Copy link
Collaborator Author

@uyarn 剩下的类型报错有空了一起看看怎么处理

@liweijie0812 liweijie0812 added the monorepo monorepo label Jan 29, 2025
@zhangpaopao0609
Copy link
Collaborator

zhangpaopao0609 commented Jan 31, 2025

@uyarn 剩下的类型报错有空了一起看看怎么处理

@liweijie0812 @uyarn
这里的 ci 修改在下面 pr 有做更新哈,然后类型报错也有处理的。当前这个 pr 就不需要了
#4974
不过暂时只做了 pull-request.yml 的更新,其它的还需要我们讨论处理一下

@zhangpaopao0609
Copy link
Collaborator

zhangpaopao0609 commented Jan 31, 2025

@uyarn 剩下的类型报错有空了一起看看怎么处理

@liweijie0812 @uyarn 这里的 ci 修改在下面 pr 有做更新哈,然后类型报错也有处理的。当前这个 pr 就不需要了 #4974 不过暂时只做了 pull-request.yml 的更新,其它的还需要我们讨论处理一下

不知道是什么原因,当初写 jsx.d.ts 的同学全量覆盖了 HTMLAttributes 类型,导致jsx随意使用属性都可以,比如 className 等,实际上没必要覆盖的,覆盖了不仅没办法类型校验,同时类型提示也没有了,后面要改一下

@uyarn
Copy link
Collaborator

uyarn commented Jan 31, 2025

@uyarn 剩下的类型报错有空了一起看看怎么处理

@liweijie0812 这个改动一起放到#4974里一起改动合适一些,ci改成pnpm 需要配合项目本身pnpm的改动一起,你把这部分改动也推到#4974的PR里呢?

@zhangpaopao0609
Copy link
Collaborator

liweijie0812

我把它单独移到 #4976

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