Join GitHub today
Refactor cron provisioning #730
Removes the flag to check if a site has schedule to trigger the service restart. Now it makes the check directly and its more expressive.
Also, it inlines the script to restart the service.
@svpernova09 there is something that bugs me, here is my scenario:
With the current implementation, homestead will not restart the cron service when the schedule has been removed. It should?
If the answer is yes, then we should restart every time, it will be very cheap because it will happen only once per provisioning. We already have precedent for this with the nginx and php-fpm services that are restarted no matter what in each provisioning cycle.
If the answer is no, then no further action is required.