search

fntneves / laravel-transactional-events

Transaction-aware Event Dispatcher for Laravel
MIT License
314 stars 28 forks source link

[1.8] Fix type on constructor #46

Closed mfn closed 4 years ago

mfn commented 4 years ago

Should fix https://github.com/fntneves/laravel-transactional-events/issues/45

Since we "loosen" the constructor args and constructors aren't subject to LSP, I think this should be good for 1.8 and then be merged into master/2.x

fntneves commented 4 years ago

Thank you @mfn for this PR!

You're an awesome contributor!