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

fix: handle some null value branch #2323

Merged
merged 4 commits into from Jan 31, 2022
Merged

Conversation

SevenOutman
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Jan 22, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

rsuite-v4 – ./docs

🔍 Inspect: https://vercel.com/rsuite/rsuite-v4/F2AEuKKvkrvba6Q8eLSMmeisPKvj
✅ Preview: Canceled

[Deployment for 1c72ffa canceled]

rsuite-nextjs – ./docs

🔍 Inspect: https://vercel.com/rsuite/rsuite-nextjs/74sYFSoh6ecgGjAn6eeXJxJqqG2c
✅ Preview: https://rsuite-nextjs-git-style-suppress-eslint-warnings-rsuite.vercel.app

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 22, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 1c72ffa:

Sandbox Source
rsuite-tp-ci Configuration

@codecov
Copy link

codecov bot commented Jan 22, 2022

Codecov Report

Merging #2323 (1c72ffa) into main (0c93d15) will decrease coverage by 0.15%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2323      +/-   ##
==========================================
- Coverage   85.82%   85.66%   -0.16%     
==========================================
  Files         274      274              
  Lines        8593     8611      +18     
  Branches     2384     2401      +17     
==========================================
+ Hits         7375     7377       +2     
- Misses        655      658       +3     
- Partials      563      576      +13     
Flag Coverage Δ
ChromeCi 85.64% <75.00%> (-0.16%) ⬇️
Firefox 85.64% <75.00%> (-0.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/DateRangePicker/DateRangePicker.tsx 87.50% <50.00%> (ø)
src/InputPicker/InputAutosize.tsx 80.32% <50.00%> (-1.34%) ⬇️
src/Uploader/UploadTrigger.tsx 73.07% <50.00%> (-1.44%) ⬇️
src/CheckTreePicker/CheckTreePicker.tsx 87.10% <60.00%> (-0.59%) ⬇️
src/InputPicker/InputPicker.tsx 86.51% <66.66%> (-0.33%) ⬇️
src/List/ListItem.tsx 93.75% <66.66%> (-6.25%) ⬇️
src/Picker/PickerOverlay.tsx 95.83% <66.66%> (-4.17%) ⬇️
src/utils/useElementResize.ts 88.88% <66.66%> (-11.12%) ⬇️
src/utils/treeUtils.ts 69.34% <71.42%> (-1.15%) ⬇️
src/CheckTreePicker/utils.ts 81.94% <75.00%> (ø)
... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c93d15...1c72ffa. Read the comment docs.

Co-authored-by: Simon Guo <simonguo.2009@gmail.com>
@SevenOutman SevenOutman merged commit 3ffd1d9 into main Jan 31, 2022
@SevenOutman SevenOutman deleted the style/suppress-eslint-warnings branch January 31, 2022 05:06
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.

None yet

2 participants