Make weave_folder re-throw at end if any .jmd failed

Per-file ProcessFailedException was previously caught and only @error'd,
so benchmark.jl always exited 0 even when one or more .jmd files
actually failed — CI showed green despite real failures (e.g. the
ComplicatedPDE/Filament.jmd ADTypes missing-dep bug masked until #1589).

Collect failures and throw at end. Per-file 'continue on failure'
behaviour is preserved (so one bad .jmd doesn't block others from
producing artifacts), but the job now exits non-zero if anything failed.

This will cause master CI to start showing red for folders that have
been silently failing — that's the intended behaviour; the goal is for
red CI to reflect real failures.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>

Author: ChrisRackauckas

src/SciMLBenchmarks.jl

@@ -89,6 +89,7 @@ function weave_folder(folder, build_list = (:script, :github))
 
     weave_files = weave_files[sortperm(priorities; rev = true)]
 
+    failures = Tuple{String,String}[]
     for file in weave_files
         try
             @eval @subprocess begin
@@ -98,8 +99,10 @@ function weave_folder(folder, build_list = (:script, :github))
         catch e
             @show folder, file
             @error(e)
+            push!(failures, (folder, file))
         end
     end
+    isempty(failures) || error("weave_folder: $(length(failures)) file(s) failed: $failures")
     return
 end