-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Check link 2 #3301
Check link 2 #3301
Conversation
LGTM, only one potential improvement on plan.py. Using setdefault function may avoid using functions like add_extract_check. |
Thanks, much simpler! |
Current coverage is 71.3% (diff: 92.1%)@@ master #3301 diff @@
==========================================
Files 78 78
Lines 9831 9929 +98
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 6994 7078 +84
- Misses 2837 2851 +14
Partials 0 0
|
superseded by #3571 |
Hi there, thank you for your contribution to Conda! This pull request has been automatically locked since it has not had recent activity after it was closed. Please open a new issue or pull request if needed. |
Supercedes #3034 "Check link". Modified checks into instructions that are now a part of the operation order.
Possible solution to #1733 by always checking permissions before linking and unlinking.