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

Use Prestissimo to download files if available #60

Merged
merged 2 commits into from
May 5, 2017

Conversation

webflo
Copy link
Member

@webflo webflo commented Mar 24, 2017

PR for #42. We don't need Prestissimo a dependency of the project, its still possible to use it because its accessible via composers autoloader.

@webflo
Copy link
Member Author

webflo commented Mar 24, 2017

Ping @oxyc and @greg-1-anderson for reviews. Thanks!

@webflo
Copy link
Member Author

webflo commented Mar 24, 2017

Did a benchmark during Drupal Dev Days.

Old: ~18 sec
New: ~3 sec

@greg-1-anderson
Copy link
Collaborator

Wow, awesome! ⭐ ⭐ ⭐ ⭐ ⭐

Haven't tried it yet, but +1 on the implementation.

@webflo webflo merged commit 1374e10 into master May 5, 2017
@webflo webflo deleted the prestissimo-curlmutli branch May 5, 2017 21:26
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.

2 participants