Skip to content

Commit dd6086d

Browse files
committed
[NFC] Remove pointless variable
`jobs` is already clamped to be no higher than the number of tests so `min(job, length(tests))` is redundant
1 parent 40f56ed commit dd6086d

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

src/ParallelTestRunner.jl

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -835,8 +835,7 @@ function runtests(mod::Module, args::ParsedArgs;
835835
jobs = something(args.jobs, default_njobs())
836836
jobs = clamp(jobs, 1, length(tests))
837837
println(stdout, "Running $jobs tests in parallel. If this is too many, specify the `--jobs=N` argument to the tests, or set the `JULIA_CPU_THREADS` environment variable.")
838-
nworkers = min(jobs, length(tests))
839-
workers = fill(nothing, nworkers)
838+
workers = fill(nothing, jobs)
840839

841840
t0 = time()
842841
results = []

0 commit comments

Comments
 (0)