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

Button Component - Introduced ForwardRef #164

Merged
merged 2 commits into from
Oct 6, 2019

Conversation

priscilawebdev
Copy link
Contributor

Type: Refactor

The following has been addressed in the PR: #162

Unit or Functional tests are included in the PR? Not yet

Description:

Wrapped Material UI Avatar component with ForwardRef

@codecov
Copy link

codecov bot commented Oct 6, 2019

Codecov Report

Merging #164 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #164      +/-   ##
==========================================
+ Coverage   82.97%   83.01%   +0.03%     
==========================================
  Files          97       98       +1     
  Lines         934      936       +2     
  Branches      149      149              
==========================================
+ Hits          775      777       +2     
  Misses        146      146              
  Partials       13       13
Impacted Files Coverage Δ
...mponents/RegistryInfoDialog/RegistryInfoDialog.tsx 66.66% <ø> (ø) ⬆️
src/components/Login/Login.tsx 91.3% <ø> (ø) ⬆️
src/components/Header/Header.tsx 89.47% <ø> (ø) ⬆️
src/components/Help/Help.tsx 100% <ø> (ø) ⬆️
src/muiComponents/Button/Button.tsx 100% <100%> (ø)

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 9eb698f...5082b09. Read the comment docs.

Copy link
Member

@juanpicado juanpicado left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@juanpicado juanpicado merged commit 909a8d9 into master Oct 6, 2019
@delete-merged-branch delete-merged-branch bot deleted the refactor/162_add_forwardRef_button branch October 6, 2019 15:17
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