-
Notifications
You must be signed in to change notification settings - Fork 60
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
ENH: First pass at dry-run #59
Conversation
Hello @ericdill! Thanks for updating the PR.
Comment last updated on September 19, 2017 at 11:29 Hours UTC |
Codecov Report
@@ Coverage Diff @@
## master #59 +/- ##
==========================================
+ Coverage 96.62% 96.88% +0.26%
==========================================
Files 2 2
Lines 237 257 +20
==========================================
+ Hits 229 249 +20
Misses 8 8
Continue to review full report at Codecov.
|
conda_mirror/conda_mirror.py
Outdated
info = resp.get('info', {}) | ||
packages = resp.get('packages', {}) | ||
# Patch the repodata.json so that all package info dicts contain a "subdir" key. | ||
# Apparently some channels on anaconda.org do not contain the 'subdir' field. I |
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.
Missing word: think
Other than the one very minor nit, it looks fine. |
Thanks @parente |
Pretty simple addition:
--dry-run
. This exits after figuring out what local packages are blacklisted and need to be removed.Closes #39