timholy / SnoopCompile.jl

Provide insights about latency (TTFX) for Julia packages
https://timholy.github.io/SnoopCompile.jl/dev/
Other
309 stars 48 forks source link

Fix undefined `SnoopPrecompile` #287

Closed rikhuijzer closed 2 years ago

rikhuijzer commented 2 years ago

Fixes

ERROR: LoadError: UndefVarError: SnoopPrecompile not defined
Stacktrace:
 [1] macro expansion
   @ ~/.julia/packages/SnoopPrecompile/bsz6N/src/SnoopPrecompile.jl:71 [inlined]
 [2] macro expansion
[...]

when loading the module via using SnoopPrecompile: @precompile_setup, @precompile_all_calls.

codecov[bot] commented 2 years ago

Codecov Report

Merging #287 (12884e8) into master (0f3db0c) will increase coverage by 0.10%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #287      +/-   ##
==========================================
+ Coverage   82.46%   82.57%   +0.10%     
==========================================
  Files          17       17              
  Lines        1945     1945              
==========================================
+ Hits         1604     1606       +2     
+ Misses        341      339       -2     
Impacted Files Coverage Δ
SnoopPrecompile/src/SnoopPrecompile.jl 85.71% <100.00%> (ø)
src/parcel_snoopi_deep.jl 88.21% <0.00%> (+0.22%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

timholy commented 2 years ago

Thanks!