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 indentation in uinvalidated docstring #235

Closed nalimilan closed 3 years ago

nalimilan commented 3 years ago

BTW, I'm not sure why the umis = part is there (the docstring doesn't refer to it).

codecov[bot] commented 3 years ago

Codecov Report

Merging #235 (3287b02) into master (484231c) will decrease coverage by 65.42%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #235       +/-   ##
===========================================
- Coverage   84.51%   19.09%   -65.43%     
===========================================
  Files           9        9               
  Lines        1589     1561       -28     
===========================================
- Hits         1343      298     -1045     
- Misses        246     1263     +1017     
Impacted Files Coverage Δ
src/invalidations.jl 0.00% <ø> (-85.61%) :arrow_down:
src/parcel_snoopl.jl 0.00% <0.00%> (-89.48%) :arrow_down:
src/parcel_snoopi_deep.jl 0.00% <0.00%> (-87.14%) :arrow_down:
src/deep_demos.jl 0.00% <0.00%> (-57.90%) :arrow_down:
src/SnoopCompile.jl 66.66% <0.00%> (-33.34%) :arrow_down:
src/write.jl 85.71% <0.00%> (-1.25%) :arrow_down:
src/parcel_snoopi.jl 92.95% <0.00%> (-1.24%) :arrow_down:
src/parcel_snoopc.jl 75.47% <0.00%> (-1.11%) :arrow_down:

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 484231c...3287b02. Read the comment docs.

timholy commented 3 years ago

Thanks! I'd be happy removing it if you prefer, but I'll merge this first.

nalimilan commented 3 years ago

No idea, I just thought I would mention it. :-)