-
Notifications
You must be signed in to change notification settings - Fork 497
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
Conversation
@uyarn 剩下的类型报错有空了一起看看怎么处理 |
@liweijie0812 @uyarn |
不知道是什么原因,当初写 jsx.d.ts 的同学全量覆盖了 HTMLAttributes 类型,导致jsx随意使用属性都可以,比如 className 等,实际上没必要覆盖的,覆盖了不仅没办法类型校验,同时类型提示也没有了,后面要改一下 |
@liweijie0812 这个改动一起放到#4974里一起改动合适一些,ci改成pnpm 需要配合项目本身pnpm的改动一起,你把这部分改动也推到#4974的PR里呢? |
我把它单独移到 #4976 了 |
🤔 这个 PR 的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
📝 更新日志
fix(组件名称): 处理问题或特性描述 ...
本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单