Skip to content
This repository was archived by the owner on Feb 12, 2019. It is now read-only.

Added a DownloadArtifacts method. #32

Merged
merged 0 commits into from
Jan 8, 2013

Conversation

alexanderfast
Copy link
Contributor

Hi again.

I've been thinking some more about the artifacts downloading and figured that there would be three things that most (all?) users would have to do:

  • Convert the artifact url to a local filename.
  • Recursively create subfolders as needed.
  • Move the temporary downloaded file.

So I added a new method that does all that, with optional flattening of the folder structure.

@stack72
Copy link
Owner

stack72 commented Nov 23, 2012

if you havent, can you add yourself as a contributor at the bottom of the readme? i want to make sure you are credited for the work you have given

paul

@alexanderfast
Copy link
Contributor Author

Sure, I'll do that, thanks. :)
That will also allow me to try one of the features here in GitHub, which is that new pushes by me on the same branch are automatically added to this pull request.

@alexanderfast
Copy link
Contributor Author

Well look at that, GitHub did what I expected. I consider that method complete for now.

@alexanderfast alexanderfast merged commit 6f9441b into stack72:master Jan 8, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants