Repository navigation
feat: add shared useAllowClear hook - #820
Conversation
|
@QDyanbing is attempting to deploy a commit to the afc163's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
Walkthrough新增 Changes清除配置规范化
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Merge Risk: ⚪ Minimal · up to
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #820 +/- ##
==========================================
+ Coverage 86.94% 87.00% +0.05%
==========================================
Files 41 42 +1
Lines 1111 1116 +5
Branches 404 392 -12
==========================================
+ Hits 966 971 +5
Misses 143 143
Partials 2 2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
变更说明
为各 rc 组件提供统一的清除配置解析能力,落实 ant-design/ant-design#59553 中 util 侧的公共接口。
useAllowClear和AllowClearConfig,支持可选的clearIcon、disabled、tabIndex和label。disabled:false/undefined返回{ disabled: true },true返回{ disabled: false };对象形式保留原有字段,并将未配置的disabled补齐为false。验证
npm run tsc通过。npm run lint通过,仅有原有的 13 条警告。npm run compile通过,已核对 ES / CommonJS 类型导出和 CommonJS 入口。git diff --check通过。文档站构建未运行。后续 rc 组件接入及 antd 配置合并不在本 PR 范围内。
Summary by CodeRabbit