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
[v4] Fix(types): external module type error (CompoundedComponent) #39058
[v4] Fix(types): external module type error (CompoundedComponent) #39058
Conversation
Codecov ReportBase: 100.00% // Head: 100.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## 4.x-stable #39058 +/- ##
============================================
Coverage 100.00% 100.00%
============================================
Files 450 450
Lines 8333 8333
Branches 2539 2539
============================================
Hits 8333 8333
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
52ee67b
to
9c35b3f
Compare
Rebase please. |
9c35b3f
to
9bdf478
Compare
Done |
components/alert/index.tsx
Outdated
@@ -69,7 +69,7 @@ interface IconNodeProps { | |||
description: AlertProps['description']; | |||
} | |||
|
|||
const IconNode: React.FC<IconNodeProps> = props => { | |||
const IconNode: React.FC<IconNodeProps> = (props) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这些括号的改动能避免吗? 有点多不好review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
看看你本地prettier 设置
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个之前的 PR 有说明,是 .prettierrc ("arrowParens": "avoid") 和 rome js config 冲突
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
git hook: rome format
when pre-commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
把本地git hook 干掉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
还有一部分, 能都还原吗?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
9bdf478
to
7923109
Compare
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
#38666
💡 Background and solution
Wrong type for
CompoundedComponent
from external moduleFor V4.x
📝 Changelog
CompoundedComponent
的组建的类型报错☑️ Self-Check before Merge