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(NumberPicker): Document optimization & Upgrade testing tools and optimize use cases & rename to ts #4753

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

luolin-ck
Copy link
Collaborator

number-picker新增功能如下:

  1. TS化
    2.文档优化
  2. 测试工具升级,用例优化

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这是您为 Fusion/Next 提的第一个 pr,感谢您为 Fusion 做出的贡献,我们会尽快进行处理。

@eternalsky eternalsky linked an issue Feb 22, 2024 that may be closed by this pull request
@luolin-ck
Copy link
Collaborator Author

close #4596

@luolin-ck luolin-ck force-pushed the fix-issue-4596 branch 5 times, most recently from 999d488 to f5d032c Compare March 25, 2024 07:43
Copy link
Contributor

@YSMJ1994 YSMJ1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

另外合并一下 master 分支,然后执行 npm run check number-picker 来检查错误和Warning,在不改动代码逻辑的前提下,尽量修复 eslint warning

components/number-picker/types.ts Show resolved Hide resolved
components/number-picker/types.ts Outdated Show resolved Hide resolved
components/number-picker/types.ts Outdated Show resolved Hide resolved
components/number-picker/types.ts Outdated Show resolved Hide resolved
components/number-picker/types.ts Outdated Show resolved Hide resolved
components/number-picker/__docs__/demo/editable/index.tsx Outdated Show resolved Hide resolved
components/number-picker/__docs__/demo/limit/index.tsx Outdated Show resolved Hide resolved
components/number-picker/__docs__/demo/precision/index.tsx Outdated Show resolved Hide resolved
components/number-picker/__docs__/demo/step/index.tsx Outdated Show resolved Hide resolved
components/number-picker/__docs__/demo/trigger/index.tsx Outdated Show resolved Hide resolved
@luolin-ck luolin-ck force-pushed the fix-issue-4596 branch 3 times, most recently from a20a3dc to eca22de Compare March 27, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

【Technical upgrade】NumberPicker
3 participants