Skip to content

Commit 6ea338d

Browse files
Fix Hwloc handling for Windows CI
The previous fix didn't properly handle the Windows case where ifelse(Sys.iswindows(), 0, ...) would return 0, causing the min() to always select 0, leading to an empty string parse error. Now we ensure at least 1 core even on Windows by wrapping the entire ifelse expression with max(..., 1). 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 7edc9dd commit 6ea338d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/runtests.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ const RETESTITEMS_NWORKERS = if GROUP == "wrappers"
2323
else
2424
parse(
2525
Int, get(ENV, "RETESTITEMS_NWORKERS",
26-
string(min(ifelse(Sys.iswindows(), 0, max(Hwloc.num_physical_cores(), 1)), 4))
26+
string(min(max(ifelse(Sys.iswindows(), 0, Hwloc.num_physical_cores()), 1), 4))
2727
)
2828
)
2929
end

0 commit comments

Comments
 (0)