-
Notifications
You must be signed in to change notification settings - Fork 35
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
samefile: Cache FileInformation for the target #409
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #409 +/- ##
==========================================
- Coverage 65.59% 65.55% -0.05%
==========================================
Files 34 34
Lines 3941 3942 +1
Branches 901 901
==========================================
- Hits 2585 2584 -1
- Misses 1001 1002 +1
- Partials 355 356 +1 ☔ View full report in Codecov by Sentry. |
a1d9456
to
fd8a243
Compare
GNU testsuite comparison:
|
@hanbings wdyt ? :) |
fd8a243
to
5eae7e3
Compare
GNU testsuite comparison:
|
And don't follow symlinks.
5eae7e3
to
7c8f6f3
Compare
Commit 7c8f6f3 has GNU testsuite comparison:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks good to me, thanks! :)
And don't follow symlinks.