Skip to content

Commit 3b5cd91

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 e57f0c8 commit 3b5cd91

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
@@ -829,8 +829,7 @@ function runtests(mod::Module, args::ParsedArgs;
829829
jobs = something(args.jobs, default_njobs())
830830
jobs = clamp(jobs, 1, length(tests))
831831
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.")
832-
nworkers = min(jobs, length(tests))
833-
workers = fill(nothing, nworkers)
832+
workers = fill(nothing, jobs)
834833

835834
t0 = time()
836835
results = []

0 commit comments

Comments
 (0)