laravel-pseudo-daemon icon indicating copy to clipboard operation
laravel-pseudo-daemon copied to clipboard

Some suggestions

Open bilogic opened this issue 3 years ago • 1 comments

Hi,

Gotta give it to you for this package! :) A few things came across my mind:

  1. I think it is better to have Daemon classes 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.
  2. 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.
  3. Technically, it is still a daemon, just that the way it is started is "different", pseudo got 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!

bilogic avatar Oct 13 '22 07:10 bilogic

Ok, apparently, my way won't work because only ->command(...) and ->exec(...) can runInBackground()

bilogic avatar Oct 13 '22 08:10 bilogic