topdown / VVV-Dashboard

Dashboard for Varying Vagrant Vagrants https://github.com/Varying-Vagrant-Vagrants/VVV
253 stars 40 forks source link

Increase the depth of the host search. #3

Closed EHLOVader closed 10 years ago

EHLOVader commented 10 years ago

Ran into issues where the configuration I used for project management based on lkwdwrd/vvv-autop-setup was not being seen by your dashboard. Here I have increased the depth which we take the search to match that of VVV

EHLOVader commented 10 years ago

Thanks.

I am not sure if it would be worth looking into reasons behind why anything over depth of 2 hits performance so hard, or what could be done to match the configuration exactly of VVV (I started this pull-request with 5, but had to change it)

Perhaps using shell commands instead of an iterator.

topdown commented 10 years ago

Its going to cause performance issues if set to high, just because of the resources required for PHP to search directories. Your actually the first to require more than 1 deep.

So I think we can leave it at 2 now and see what happens. Thanks