Closed avagin closed 3 months ago
Attention: Patch coverage is 62.50000%
with 42 lines
in your changes are missing coverage. Please review.
Project coverage is 70.20%. Comparing base (
cb39c62
) to head (4bf16da
).:exclamation: Current head 4bf16da differs from pull request most recent head 8e83b7a. Consider uploading reports for the commit 8e83b7a to get more accurate results
Files | Patch % | Lines |
---|---|---|
criu/cr-check.c | 62.50% | 42 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Looks good.
minor nits:
Do we want to add it to kdat?
A friendly reminder that this PR had no activity for 30 days.
Check that the file device and inode shown in /proc/pid/maps match values returned by stat(2).