Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Font Awesome integration #16

Closed
wants to merge 14 commits into from
Closed

Add Font Awesome integration #16

wants to merge 14 commits into from

Conversation

nagacoder
Copy link

@nagacoder nagacoder commented Apr 20, 2019

What: Add Font Awesome integration

Why:

How:

Checklist:

  • Tests
  • Ready to be merged

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Apr 20, 2019

Codecov Report

Merging #16 into master will decrease coverage by 42.1%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #16       +/-   ##
===========================================
- Coverage     100%   57.89%   -42.11%     
===========================================
  Files           4        4               
  Lines          19       19               
  Branches        3        3               
===========================================
- Hits           19       11        -8     
- Misses          0        8        +8
Impacted Files Coverage Δ
src/components/layout.js 0% <ø> (-100%) ⬇️
src/components/image.js 0% <0%> (-100%) ⬇️
src/components/seo.js 87.5% <0%> (-12.5%) ⬇️

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 55f5595...baf506d. Read the comment docs.

Copy link
Contributor

@zainfathoni zainfathoni left a comment

Choose a reason for hiding this comment

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

Please sign all your commits, otherwise, this PR will never be merged.
I've set the branch configuration to always require commit signing.

You can do it by cherrypick-ing your commits and signing the new commits.

@zainfathoni zainfathoni changed the title add font-awesome Add Font Awesome integration Apr 20, 2019
@zainfathoni zainfathoni added the enhancement New feature or request label Apr 20, 2019
@zainfathoni zainfathoni added this to In progress in Infrastructure via automation Apr 20, 2019
@zainfathoni zainfathoni added this to the MVP Launching milestone Apr 20, 2019
@r17x r17x mentioned this pull request Apr 20, 2019
43 tasks
@nagacoder nagacoder force-pushed the feature/fontAwesome branch 2 times, most recently from 77e2304 to baf506d Compare April 21, 2019 07:26
@nagacoder nagacoder closed this Apr 21, 2019
Infrastructure automation moved this from In progress to Done Apr 21, 2019
@nagacoder nagacoder deleted the feature/fontAwesome branch April 21, 2019 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants