Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

feat(Icon): Adding lightning icon for Teams theme#1385

Merged
notandrew merged 4 commits intomasterfrom
feature/icon-lightning
May 24, 2019
Merged

feat(Icon): Adding lightning icon for Teams theme#1385
notandrew merged 4 commits intomasterfrom
feature/icon-lightning

Conversation

@notandrew
Copy link
Contributor

Adding lightning icon:

image

@codecov
Copy link

codecov bot commented May 23, 2019

Codecov Report

Merging #1385 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1385      +/-   ##
==========================================
+ Coverage   73.47%   73.47%   +<.01%     
==========================================
  Files         777      778       +1     
  Lines        5858     5859       +1     
  Branches     1706     1726      +20     
==========================================
+ Hits         4304     4305       +1     
  Misses       1548     1548              
  Partials        6        6
Impacted Files Coverage Δ
...emes/teams/components/Icon/svg/icons/lightning.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 35d1deb...edba81d. Read the comment docs.

@kuzhelov
Copy link
Contributor

kuzhelov commented May 24, 2019

looks good to me :) just one thing to ensure before merge - was this icon verified by designers? @codepretty

@codepretty codepretty changed the title Adding lightning icon feat(Icon): Adding lightning icon for Teams theme May 24, 2019
@notandrew notandrew merged commit 9a34737 into master May 24, 2019
@delete-merged-branch delete-merged-branch bot deleted the feature/icon-lightning branch May 24, 2019 20:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants