-
-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: unix_sockname: use tmp_path #3555
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3555 +/- ##
=======================================
Coverage 97.91% 97.91%
=======================================
Files 43 43
Lines 8577 8577
Branches 1384 1384
=======================================
Hits 8398 8398
Misses 74 74
Partials 105 105 Continue to review full report at Codecov.
|
2 similar comments
Codecov Report
@@ Coverage Diff @@
## master #3555 +/- ##
=======================================
Coverage 97.91% 97.91%
=======================================
Files 43 43
Lines 8577 8577
Branches 1384 1384
=======================================
Hits 8398 8398
Misses 74 74
Partials 105 105 Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #3555 +/- ##
=======================================
Coverage 97.91% 97.91%
=======================================
Files 43 43
Lines 8577 8577
Branches 1384 1384
=======================================
Hits 8398 8398
Misses 74 74
Partials 105 105 Continue to review full report at Codecov.
|
Yes, the intention is replacing |
Thanks! |
Ref: #3551
The intention from/in #3551 is not really clear to me.
Mainly via #3553 (comment), but should be done in other places then also?
@asvetlov
Please consider picking it up.