-
Notifications
You must be signed in to change notification settings - Fork 73
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
support for resolved symlinks in copytree #230
support for resolved symlinks in copytree #230
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #230 +/- ##
==========================================
- Coverage 99.07% 98.99% -0.08%
==========================================
Files 24 24
Lines 5085 5095 +10
==========================================
+ Hits 5038 5044 +6
- Misses 47 51 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Just an FYI the Samba target doesn't support symlinks so you'll need to only run the symlink tests on Windows. |
f3c0d02
to
ae8e4f4
Compare
38cb37e
to
baff1df
Compare
baff1df
to
13ccd47
Compare
13ccd47
to
1cadb72
Compare
@brno32, @jborean93: Any progress on it? |
@brno32: Why have you closed this PR? |
@jborean93: Can you look? |
It's up to @brno32 if they wish to continue with the PR or not. I assume they either don't have the time to contribute anymore or lost interest in this particular feature, both of which are perfectly fine. |
Fixes #221