search

JuliaLang / Distributed.jl

Create and control multiple Julia processes remotely for distributed computing. Ships as a Julia stdlib.
https://docs.julialang.org/en/v1/stdlib/Distributed/
MIT License
20 stars 8 forks source link

Guard rmprocs in tests to avoid warning. Fix leaky stderr from SIGTERM test. #93

Closed IanButterworth closed 4 months ago

IanButterworth commented 4 months ago

Fixes 

  | ┌ Warning: rmprocs: process 1 not removed
  | └ @ Distributed /Users/julia/.julia/scratchspaces/a66863c6-20e8-4ff4-8a62-49f30b1f605e/agent-cache/default-honeycrisp-XC9YQX9HH2.0/build/default-honeycrisp-XC9YQX9HH2-0/julialang/julia-master/julia-83f908a311/share/julia/stdlib/v1.12/Distributed/src/cluster.jl:1049

Fixes https://github.com/JuliaLang/Distributed.jl/issues/15

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.53%. Comparing base (8c03305) to head (41cd14f).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #93 +/- ## ========================================== - Coverage 79.34% 71.53% -7.81% ========================================== Files 10 10 Lines 1951 1950 -1 ========================================== - Hits 1548 1395 -153 - Misses 403 555 +152 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

IanButterworth commented 4 months ago

I'd just merge this but PRs require 1 review approval

IanButterworth commented 4 months ago

Thanks. I just invoked a Distributed bump