Skip to content
This repository has been archived by the owner on Jan 16, 2022. It is now read-only.

List Component - Introduced ForwardRef #181

Merged
merged 6 commits into from
Oct 12, 2019

Conversation

priscilawebdev
Copy link
Contributor

Type: Refactor

The following has been addressed in the PR: #162

Description:

Wrapped Material UI List Component with ForwardRef.

@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #181 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #181      +/-   ##
=========================================
+ Coverage   88.87%   88.9%   +0.02%     
=========================================
  Files         116     117       +1     
  Lines         926     928       +2     
  Branches      163     147      -16     
=========================================
+ Hits          823     825       +2     
- Misses         89      90       +1     
+ Partials       14      13       -1
Impacted Files Coverage Δ
src/components/Engines/Engines.tsx 100% <ø> (ø) ⬆️
src/components/ActionBar/ActionBar.tsx 84.37% <ø> (ø) ⬆️
src/components/Versions/VersionsTagList.tsx 100% <ø> (ø) ⬆️
src/components/Install/Install.tsx 85.71% <ø> (ø) ⬆️
src/components/Author/Author.tsx 90.9% <ø> (ø) ⬆️
src/components/Dist/Dist.tsx 88.88% <ø> (ø) ⬆️
src/components/UpLinks/UpLinks.tsx 88.88% <ø> (ø) ⬆️
src/components/DetailSidebar/DetailSidebar.tsx 85% <ø> (ø) ⬆️
src/components/Repository/Repository.tsx 100% <ø> (ø) ⬆️
src/components/Package/styles.ts 100% <ø> (ø) ⬆️
... and 4 more

juanpicado
juanpicado previously approved these changes Oct 12, 2019
@juanpicado juanpicado merged commit 0c4fb7d into master Oct 12, 2019
@delete-merged-branch delete-merged-branch bot deleted the refactor/162_introduce_forwardRef_list branch October 12, 2019 20:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants