korsbo / Latexify.jl

Convert julia objects to LaTeX equations, arrays or other environments.
MIT License
562 stars 59 forks source link

Adjustment #218

Closed gustaphe closed 2 years ago

gustaphe commented 2 years ago

Allow vectors of adjustment symbols, and make adjustment affect mdtable.

codecov[bot] commented 2 years ago

Codecov Report

Merging #218 (e0d3aa2) into master (d1845a5) will increase coverage by 0.55%. The diff coverage is 95.23%.

@@            Coverage Diff             @@
##           master     #218      +/-   ##
==========================================
+ Coverage   62.68%   63.24%   +0.55%     
==========================================
  Files          23       23              
  Lines         812      827      +15     
==========================================
+ Hits          509      523      +14     
- Misses        303      304       +1     
Impacted Files Coverage Δ
src/mdtable.jl 88.88% <90.90%> (ø)
src/latexarray.jl 96.15% <100.00%> (+0.50%) :arrow_up:
src/latextabular.jl 92.30% <100.00%> (+0.64%) :arrow_up:

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 d1845a5...e0d3aa2. Read the comment docs.

korsbo commented 2 years ago

This looks fine to me. You can go ahead and stop supporting Julia 0.7 - the only reason I have not done that myself is that it has just kept working.

Be sure to bump the Latexify version though.

gustaphe commented 2 years ago

This way was cleaner if it passes CI. Otherwise I might just do that.