search

LordSimal / cakephp-scheduler

CakePHP scheduler plugin
MIT License
6 stars 2 forks source link

propose: adding exception handler #12

Closed crhdev closed 5 days ago

crhdev commented 2 weeks ago

Hi! I was playing a little with the scheduler, it works great for small projects. An improvement I should have implemented was exception handling when a command fails. Could something like this be integrated?

https://github.com/crhdev/cakephp-scheduler/blob/crhdev-patch-exception_handler/src/Command/ScheduleRunCommand.php#L52

LordSimal commented 1 week ago

yes, this would be a good idea. I am just not sure if using the event system to react to exceptions is the correct way here... gotta have to think about that.