search

squigg / azure-queue-laravel

PHP Laravel Queue Driver package for Microsoft Azure Storage Queues
43 stars 24 forks source link

Laravel 8 Support #14

Closed aaronhuisinga closed 2 years ago

aaronhuisinga commented 4 years ago

Can we get a release tagged for Laravel 8? I believe through my testing the package should work fine with the Laravel 8 queue.

squigg commented 4 years ago

I'm looking at this now but it's not straightforward. The dependency on microsoft/azure-storage-queue requires Guzzle ~6.0, but Laravel has upgraded to Guzzle ~7.0. There are a number of open issues on the repo but nobody seems to be actively looking at it.

I am going through the breaking changes for Guzzle 6->7 to see if I can submit a PR for the Microsoft package to get a version published that we can use. If this doesn't go anywhere then I can temporarily use a fork. So far it looks like it will be compatible but I need to do more testing.

squigg commented 4 years ago

Looks like we can work around this in the short term with an inline alias for Guzzle in composer.json. I have added this to the docs and will merge and tag PR #15.

Done: https://github.com/squigg/azure-queue-laravel/releases/tag/v8.0.0

I'll leave this open to stay up to date with progress on the upstream package.

MrThePlague commented 3 years ago

Looks like Microsoft just pushed the changes required to implement Laravel 8 support!