We're getting close to finishing so I'll expand. While fixing a bug on the rector-laravel package, I find out that you can queue closures using dispatch (as per the docs) but you can't when using Bus::dispatch(). I thought this was a bug but I got the issue closed (probably for not being assertive enough, haha). What do you think?
@j3j5 I actually managed to get a PR merged recently, but yeah, you gotta have quite thick skin (and be quite deferential, which doesn’t come naturally to me 😬)
@michael yeah, not my first rodeo and I keep reporting making PRs because I still benefit from them. I can understand from the other side, a repo like that isn't easy to manage, but that's why I try to make the issues and PRs as clear as possible. I think in this case Dries didn't read past the "not 100% sure if it's a bug" 🤷
Add comment