Skip to content

Conversation

@coderbaozi
Copy link

@vercel
Copy link

vercel bot commented May 21, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
picker ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2023 8:48am

@yoyo837
Copy link
Member

yoyo837 commented May 21, 2023

Please add some test case for the fix.

@BoyYangzai
Copy link
Contributor

BoyYangzai commented May 22, 2023

@coderbaozi
image
测试用例没有完全通过哈 麻烦再检查下

@coderbaozi
Copy link
Author

测试snapshot是不是有更新的pr没合并,我现在跑的结果和未修改的源代码的测试结果一样qaq

@codecov
Copy link

codecov bot commented May 23, 2023

Codecov Report

Merging #637 (1c31f02) into master (85a8c5b) will decrease coverage by 0.13%.
The diff coverage is 57.14%.

❗ Current head 1c31f02 differs from pull request most recent head 139279d. Consider uploading reports for the commit 139279d to get more accurate results

@@            Coverage Diff             @@
##           master     #637      +/-   ##
==========================================
- Coverage   99.21%   99.08%   -0.13%     
==========================================
  Files          52       52              
  Lines        2407     2413       +6     
  Branches      711      714       +3     
==========================================
+ Hits         2388     2391       +3     
- Misses         17       20       +3     
  Partials        2        2              
Impacted Files Coverage Δ
src/RangePicker.tsx 98.56% <57.14%> (-0.71%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@BoyYangzai
Copy link
Contributor

@coderbaozi 补充一下 test case~

@BoyYangzai
Copy link
Contributor

@yoyo837 @zombieJ
看下这两个 pr
#639

@yoyo837
Copy link
Member

yoyo837 commented May 24, 2023

感谢贡献。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RangePicker设置 disabled=[false, true] 之后,第二项还是能改

3 participants