Skip to content

Commit 3991f99

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 dc73e6d commit 3991f99

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
@@ -832,8 +832,7 @@ function runtests(mod::Module, args::ParsedArgs;
832832
jobs = something(args.jobs, default_njobs())
833833
jobs = clamp(jobs, 1, length(tests))
834834
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.")
835-
nworkers = min(jobs, length(tests))
836-
workers = fill(nothing, nworkers)
835+
workers = fill(nothing, jobs)
837836

838837
t0 = time()
839838
results = []

0 commit comments

Comments
 (0)