Skip to content

Chart color rebrand#1024

Merged
jondlm merged 2 commits intomasterfrom
chart-color-rebrand
Aug 19, 2019
Merged

Chart color rebrand#1024
jondlm merged 2 commits intomasterfrom
chart-color-rebrand

Conversation

@jondlm
Copy link
Copy Markdown
Contributor

@jondlm jondlm commented Aug 19, 2019

PR Checklist

  • Manually tested across supported browsers
    • Chrome
    • Firefox
    • Safari
    • Edge (Win 10)
  • Unit tests written (common at minimum)
  • PR has one of the semver- labels
  • Two core team engineer approvals
  • One core team UX approval

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1024 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1024   +/-   ##
=======================================
  Coverage   90.21%   90.21%           
=======================================
  Files         186      186           
  Lines        3721     3721           
  Branches     1234     1234           
=======================================
  Hits         3357     3357           
  Misses        341      341           
  Partials       23       23
Impacted Files Coverage Δ
src/components/BarChart/BarChart.jsx 84.09% <ø> (ø) ⬆️
src/components/PieChart/PieChart.jsx 100% <ø> (ø) ⬆️
src/components/LineChart/LineChart.jsx 70.63% <ø> (ø) ⬆️
src/components/Points/Points.jsx 95.65% <ø> (ø) ⬆️
src/components/Bars/Bars.jsx 90.24% <ø> (ø) ⬆️
src/components/Lines/Lines.jsx 100% <ø> (ø) ⬆️

@jondlm jondlm merged commit b3bc023 into master Aug 19, 2019
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.

3 participants