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

SvgIcon Component - Introduced ForwardRef #184

Merged
merged 3 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 SvgIcon Component with ForwardRef.

@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #184 into master will decrease coverage by 0.08%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #184      +/-   ##
=========================================
- Coverage   88.28%   88.2%   -0.09%     
=========================================
  Files         112     113       +1     
  Lines         922     924       +2     
  Branches      147     147              
=========================================
+ Hits          814     815       +1     
- Misses         95      96       +1     
  Partials       13      13
Impacted Files Coverage Δ
src/icons/GitHub.tsx 50% <ø> (ø) ⬆️
src/muiComponents/SvgIcon/SvgIcon.tsx 50% <50%> (ø)

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 7548c89...77d05c3. Read the comment docs.

@juanpicado juanpicado merged commit 8b86ded into master Oct 12, 2019
@delete-merged-branch delete-merged-branch bot deleted the refactor/162_introduced_forwardRef_svgIcon branch October 12, 2019 09:42
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