Closed fonsp closed 6 months ago
Attention: 3 lines
in your changes are missing coverage. Please review.
Comparison is base (
43d38e9
) 37.50% compared to head (b2532bc
) 36.51%. Report is 5 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
src/PlotlyJS.jl | 50.00% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks! Could you also make a release? Then we can work on https://github.com/JuliaPlots/Plots.jl/pull/4862
Thanks again :)
This PR makes sure that the same version of the plotlyjs web asset is used for all imports: through artifact or through CDN.
This also adds
_js_version
and_js_path
as public API: this means other packages can depend on PlotlyJS.jl to get the plotly.min.js asset. Used in https://github.com/JuliaPlots/Plots.jl/pull/4862