Skip to content
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

Update conda_mirror.py #21

Merged
merged 1 commit into from
Jan 10, 2017
Merged

Update conda_mirror.py #21

merged 1 commit into from
Jan 10, 2017

Conversation

jneines
Copy link
Contributor

@jneines jneines commented Jan 10, 2017

Added an optional command line parameter to choose the temporary download location for applying the checks before moving the packages to the target location, which was hardcoded to the default temporary directory before. Now this is the default location, when not explicitly specifying the temp_directory parameter.

This should fix issue #19

Added an optional command line parameter to choose the temporary download location for applying the checks before moving the packages to the target location, which was hardcoded to the default temporary directory before. Now this is the default location, when not explicitly specifying the temp_directory parameter.
@codecov-io
Copy link

Current coverage is 92.48% (diff: 100%)

Merging #21 into master will increase coverage by 0.03%

@@             master        #21   diff @@
==========================================
  Files             2          2          
  Lines           212        213     +1   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            196        197     +1   
  Misses           16         16          
  Partials          0          0          

Powered by Codecov. Last update e201b61...332b24e

@ericdill
Copy link
Contributor

LGTM. Thanks @jneines . I'll cut a new release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants