disable precompilation for selected packages via Preferences

[t-bltg]

Fix #324.
Fix #298.
Supersedes #299.

[codecov]

Codecov Report

Base: 83.56% // Head: 83.57% // Increases project coverage by +0.01% 🎉

Coverage data is based on head (f688ea8) compared to base (333b645).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #334      +/-   ##
==========================================
+ Coverage   83.56%   83.57%   +0.01%     
==========================================
  Files          17       17              
  Lines        2184     2186       +2     
==========================================
+ Hits         1825     1827       +2     
  Misses        359      359              

Impacted Files	Coverage Δ
SnoopPrecompile/src/SnoopPrecompile.jl	94.11% <100.00%> (+0.36%)	⬆️

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[t-bltg]

Hum, ERROR: LoadError: Preferences.jl can only be used on Julia v1.6+!.

=> restricted to julia 1.6+

[timholy]

I wonder if we should make it more complicated 🙂 . Specifically, would it be useful to allow one to specify a list of packages one shouldn't execute the precompile workload for? If I'm developing a specific package I still might want good precompilation on others.

I'm happy to help with this. I was going to transfer SnoopPrecompile first but with this as a head-start there's probably no reason to do so.

[t-bltg]

If I'm developing a specific package I still might want good precompilation on others.

Good point , yes.

Specifically, would it be useful to allow one to specify a list of packages one shouldn't execute the precompile workload for?

Done.

E.g. in ~/.julia/environments/v1.9/LocalPreferences.toml

[SnoopPrecompile]
skip_precompile = ["MarchingCubes"]

Do you have an idea for crafting a test for this ?

[timholy]

I tweaked the docs since precompilation itself isn't disabled, we're just skipping the workload that results in "extra" precompilation.

As far as a test, one could perhaps use

@precompile_all_calls begin
    global workload_ran = true
end

and check isdefined(SomeModule, :workload_ran) && SomeModule.workload_ran.

[t-bltg]

@timholy, can you cancel the previous Documenter workflows ? (Or maybe give me contribution rights to this repo ?)

They seem to hang and fill up the actions queue thus preventing running CI for most recent commits.
I've added a concurrency block in the .yml files.

[timholy]

Thanks so much!

Ideally I want to try a PkgEval run before we release this, but we should get this out soon: this meets a real need! ❤️