e0f0c053

By: Tom Sydney Kerckhove <syd@cs-syd.eu>

Add includeTests option to makeWeederCheck (default false)

By default, makeWeederCheck no longer feeds weeder the test suites' .hie
files: it skips buildTestsWithoutRunning, so code used only by tests is
reported as a weed ("don't keep code alive purely by testing it").

Set includeTests = true to restore the previous behavior (compile the test
code without running it and include its .hie files), which prevents
false-positives for code that exists only to be tested.

Note: with includeTests = false the only roots are those in weeder.toml; a
library whose public API is exercised only by its own tests needs those
modules declared as roots.

Suite timing

Time to Start Worker time Duration Time to finish Idle
Config 3m01s 2s 2s 3m03s 3m01s
Eval 3m03s 2m54s 2m54s 5m58s 0s
Build 2h22m46s 3m12s 1m50s 2h24m36s 2h16m47s
Suite 3m01s 6m09s 2h21m35s 2h24m36s 2h19m49s

Timeline

0s3m20s3m40s4m4m20s4m40s5m5m20s5m40s2h23m2h23m2h23m2h24m2h24m