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

Update fonts: remove unused glyphs and Macintosh table #2171

Merged
merged 2 commits into from Feb 21, 2020

Conversation

ylemkimon
Copy link
Member

@codecov-io
Copy link

codecov-io commented Dec 12, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2171      +/-   ##
==========================================
+ Coverage   94.73%   94.74%   +<.01%     
==========================================
  Files          83       83              
  Lines        5304     5306       +2     
  Branches      927      928       +1     
==========================================
+ Hits         5025     5027       +2     
  Misses        256      256              
  Partials       23       23
Flag Coverage Δ
#screenshotter 89.2% <0%> (-0.04%) ⬇️
#test 88.46% <100%> (ø) ⬆️
Impacted Files Coverage Δ
contrib/auto-render/auto-render.js 76.36% <ø> (ø) ⬆️
src/functions/delimsizing.js 96.87% <100%> (+0.06%) ⬆️

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 31cb69f...148adc2. Read the comment docs.

Copy link
Member

@kevinbarabash kevinbarabash left a comment

Choose a reason for hiding this comment

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

Thanks again for cleaning up the fonts. Sorry for the delay in code reviewing.

@kevinbarabash kevinbarabash merged commit f7334e4 into master Feb 21, 2020
@ylemkimon ylemkimon deleted the font-unused branch July 11, 2020 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants