davibarreira / NotebookToLaTeX.jl

A Julia package for converting your Pluto and Jupyter Notebooks into beautiful Latex.
MIT License
72 stars 4 forks source link

Removing Makie, CairoMakie and Plots as requirements via Requires.jl #8

Closed davibarreira closed 2 years ago

davibarreira commented 2 years ago

This PR moves some code around in order to improve the organization overall. It also fixes some small bugs:

The main change comes from the fact that now it uses Requires.jl in order to avoid Makie, CairoMakie and Plots as dependencies. Theses packages are only necessary for the Pluto converter. Hence, if you just wish to convert Jupyter notebooks, the package precompilation time will be quite low.

codecov[bot] commented 2 years ago

Codecov Report

Merging #8 (ed36262) into master (8e25d91) will increase coverage by 0.05%. The diff coverage is 98.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   96.81%   96.86%   +0.05%     
==========================================
  Files           5        7       +2     
  Lines         408      415       +7     
==========================================
+ Hits          395      402       +7     
  Misses         13       13              
Impacted Files Coverage Δ
src/markdowntolatex.jl 99.17% <0.00%> (ø)
src/plutoconverter.jl 99.13% <99.13%> (ø)
src/NotebookToLaTeX.jl 90.90% <100.00%> (-8.00%) :arrow_down:
src/jupyterconverter.jl 100.00% <100.00%> (ø)

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 8e25d91...ed36262. Read the comment docs.