laravel-pseudo-daemon
laravel-pseudo-daemon copied to clipboard
Some suggestions
Hi,
Gotta give it to you for this package! :) A few things came across my mind:
- I think it is better to have
Daemonclasses instead of requiring every daemon to be a command, this fills up artisan with commands which might be duplicate, especially if I prefer to keep my existing commands as just commands. - Laravel's example is to call it this way
$schedule->call(new DaemonClass)->everyMinute()...;. Offhand, I think it is even backward compatible to your current approach. - Technically, it is still a daemon, just that the way it is started is "different",
pseudogot me a bit confused there, i.e. half-daemon? My suggestion here is that a clearer name can help, your brief description was quite clear though.
Useful package!
Ok, apparently, my way won't work because only ->command(...) and ->exec(...) can runInBackground()