Skip to content
This repository has been archived by the owner on Mar 15, 2020. It is now read-only.

Removed uneeded dependency on symfony/finder #16

Merged
merged 1 commit into from
Jan 5, 2016
Merged

Conversation

Nyholm
Copy link
Contributor

@Nyholm Nyholm commented Jan 2, 2016

We do never use it.

@xabbuh
Copy link

xabbuh commented Jan 3, 2016

The use statement in the Updater class should be removed too.

@@ -20,7 +20,6 @@
use Humbug\SelfUpdate\Strategy\StrategyInterface;
use Humbug\SelfUpdate\Strategy\ShaStrategy;
use Humbug\SelfUpdate\Strategy\GithubStrategy;
use Symfony\Component\Finder\Finder;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The use statement is already removed.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops :(

@sagikazarmark
Copy link

👍

@padraic
Copy link
Collaborator

padraic commented Jan 5, 2016

Merging...merging...no explosions...merging ;) Thanks!

padraic added a commit that referenced this pull request Jan 5, 2016
Removed uneeded dependency on symfony/finder
@padraic padraic merged commit 7cba1ad into humbug:master Jan 5, 2016
@webmozart
Copy link
Contributor

@padraic Thanks a lot! :)

@Nyholm Nyholm deleted the sf branch January 5, 2016 19:04
@Nyholm
Copy link
Contributor Author

Nyholm commented Jan 5, 2016

Thank you for merging! Could you tag 1.0.3?

@padraic
Copy link
Collaborator

padraic commented Jan 6, 2016

I'll be tagging shortly @Nyholm

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.

5 participants