Skip to content

Infinite Scroll#8

Draft
fsjsd wants to merge 7 commits intomainfrom
infinitescroll
Draft

Infinite Scroll#8
fsjsd wants to merge 7 commits intomainfrom
infinitescroll

Conversation

@fsjsd
Copy link
Owner

@fsjsd fsjsd commented Apr 10, 2022

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Apr 10, 2022

Codecov Report

Merging #8 (6355420) into main (60e1241) will decrease coverage by 4.55%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   96.83%   92.27%   -4.56%     
==========================================
  Files          74       77       +3     
  Lines         759      803      +44     
  Branches      165      171       +6     
==========================================
+ Hits          735      741       +6     
- Misses         24       62      +38     
Impacted Files Coverage Δ
...roll/recyclingvirtuallist/RecyclingVirtualList.tsx 0.00% <0.00%> (ø)
src/features/infinitescroll/utils/db.ts 40.00% <40.00%> (ø)
...eatures/infinitescroll/InfiniteScrollContainer.tsx 50.00% <50.00%> (-50.00%) ⬇️
src/features/infinitescroll/utils/data.ts 100.00% <100.00%> (ø)
src/features/infinitescroll/InfiniteScroll.tsx 0.00% <0.00%> (-100.00%) ⬇️

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 60e1241...6355420. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Apr 11, 2022

This pull request introduces 3 alerts when merging 006f0bc into 29dc82a - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Apr 16, 2022

This pull request introduces 3 alerts when merging e9e6247 into 9977d22 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented May 4, 2022

This pull request introduces 3 alerts when merging 6355420 into 60e1241 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants