Skip to content

Commit

Permalink
Use Base.require workaround in IJuliaExt (#4749)
Browse files Browse the repository at this point in the history
* Always use Requires for IJulia

Addresses issue #4719.

* Move non-IJulia-dependent code out of IJuliaExt.jl

Reduces code loaded by @require while handling
IJulia via Requires only.

* Handle IJulia via extension if possible, but use Base.require

* Formatting update ext/IJuliaExt.jl

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
  • Loading branch information
oschulz and github-actions[bot] authored May 26, 2023
1 parent 11dba9d commit 21b4d91
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ext/IJuliaExt.jl
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ module IJuliaExt
import Plots: @ext_imp_use, Plots, Plot
using Base64

@ext_imp_use :import IJulia
const IJulia =
Base.require(Base.PkgId(Base.UUID("7073ff75-c697-5162-941a-fcdaad2a7d2a"), "IJulia"))

function _init_ijulia_plotting()
# IJulia is more stable with local file
Expand Down

0 comments on commit 21b4d91

Please sign in to comment.