search

JuliaGraphics / FreeTypeAbstraction.jl

A Julian abstraction layer over FreeType.jl
Other
25 stars 20 forks source link

Remove SArray #68

Closed SimonDanisch closed 2 years ago

SimonDanisch commented 2 years ago

It wasn't really used, and makes things a bit simpler

codecov[bot] commented 2 years ago

Codecov Report

Merging #68 (0789347) into master (4e381e7) will not change coverage. The diff coverage is 50.00%.

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   91.26%   91.26%           
=======================================
  Files           6        6           
  Lines         332      332           
=======================================
  Hits          303      303           
  Misses         29       29
Impacted Files Coverage Δ
src/FreeTypeAbstraction.jl 100.00% <ø> (ø)
src/layout.jl 97.67% <0.00%> (ø)
src/types.jl 83.83% <66.66%> (ø)

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 a8fecea...0789347. Read the comment docs.

SimonDanisch commented 2 years ago

Ok, this change is non breaking for Makie... @t-bltg, do want to take a look as well if it breaks your projects? The coverage decrease seems bogus.

t-bltg commented 2 years ago

All good on my side, no changes !